- Notifications
You must be signed in to change notification settings - Fork262
refactor: ruff fixes and remove / reflow commented code#958
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
refactor: ruff fixes and remove / reflow commented code#958
Uh oh!
There was an error while loading.Please reload this page.
Conversation
codejedi365 commentedJun 15, 2024
From some of the changes you are adding here, I think ruff is not reading our configuration. Or if it is than your version of pre-commit does not match the ruff version in pyproject.toml. The reason I say this is that in the lint job it is running ruff but it hasn't been an issue lately. The |
wyardley commentedJun 15, 2024
Let me make sure I have the right version - should have installed the one in the project config, but maybe there’s a discrepancy somewhere? |
wyardley commentedJun 15, 2024
FWIW:
|
wyardley commentedJun 15, 2024
Maybe the updates in#957 will help? I notice that it's got the formatting one pinned back, though maybe now the two can be consolidated? Either way, I think the more usual default of the |
e650dac tof50c9e4Compare- Remove redundant literal unionhttps://docs.astral.sh/ruff/rules/redundant-literal-union/- Small adjustment to use list comprehension vs. iteration for changelog generation in test
f50c9e4 tof451590Comparewyardley commentedJun 15, 2024
@codejedi365 see if you like this better - got rid of all the autoformatting. If these two don't make sense, also feel free to just close. |
codejedi365 left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks for all the work. Not at my computer to try the ruff version changes, but thanks for adapting.
codejedi365 commentedJun 15, 2024
I agree it should operate on only the changed files |
Uh oh!
There was an error while loading.Please reload this page.
tests/fixtures/git_repo.pyruffhappy.All the changes are test related, so if you want
testvsrefactor, let me know, or feel free to adjust the type when merging.