Skip to content

feat(ci): add cairo-lint#276

Draft
dlaciport wants to merge 5 commits intostarkware-bitcoin:mainfrom
dlaciport:feat/cairo-lint
Draft

feat(ci): add cairo-lint#276
dlaciport wants to merge 5 commits intostarkware-bitcoin:mainfrom
dlaciport:feat/cairo-lint

Conversation

@dlaciport
Copy link
Copy Markdown
Contributor

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 2, 2024

@bloomingpeach is attempting to deploy a commit to the keep-starknet-strange Team on Vercel.

A member of the Team first needs to authorize it.

@b-j-roberts
Copy link
Copy Markdown
Contributor

Nice! Would you be able to fix existing lint warnings as part of this PR so the CI passes?

@dlaciport
Copy link
Copy Markdown
Contributor Author

@b-j-roberts I think the currentcairo-lint isn't ready to be integrated to the CI, while fixing I encountered several false positives

@b-j-roberts
Copy link
Copy Markdown
Contributor

Alright, thank you for looking into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants