Skip to content

loosen linting in ruff#704

Merged
wakamex merged 1 commit intomainfrom
dont_lint_too_many_arguments_in_ruff
Jul 20, 2023
Merged

loosen linting in ruff#704
wakamex merged 1 commit intomainfrom
dont_lint_too_many_arguments_in_ruff

Conversation

@wakamex
Copy link
Copy Markdown
Contributor

@wakamex wakamex commented Jul 20, 2023

No description provided.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
elf-simulations ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2023 8:59pm

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4958 🚀

Copy link
Copy Markdown
Contributor

@sentilesdal sentilesdal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure about these ignores. Should have a discussion with the team

@wakamex wakamex marked this pull request as draft July 20, 2023 20:49
Copy link
Copy Markdown
Contributor

@sentilesdal sentilesdal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is fine if just for ruff. if this becomes our linter for ci then we'll have to discuss

@wakamex wakamex marked this pull request as ready for review July 20, 2023 20:53
@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4974 🚀

@wakamex wakamex merged commit 5f4e625 into main Jul 20, 2023
@wakamex wakamex deleted the dont_lint_too_many_arguments_in_ruff branch July 20, 2023 21:03
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.

3 participants