Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Conversation

@wyardley
Copy link
Contributor

@wyardleywyardley commentedJun 15, 2024
edited
Loading

All the changes are test related, so if you wanttest vsrefactor, let me know, or feel free to adjust the type when merging.

@codejedi365
Copy link
Contributor

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... in the type definitions is one of the things that ruff has gone back and forth with. First it was on a new line and a few versions back I had to move it all to inline.

@wyardley
Copy link
ContributorAuthor

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
Copy link
ContributorAuthor

FWIW:

% ruff --versionruff 0.4.4

@wyardley
Copy link
ContributorAuthor

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 theruff pre-commit hook is to only operate on the changed files vs. all files as it's currently configured.

@codejedi365codejedi365 self-requested a reviewJune 15, 2024 16:40
@wyardleywyardleyforce-pushed thewyardley/commented_out_code branch frome650dac tof50c9e4CompareJune 15, 2024 16:53
- 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
@wyardleywyardleyforce-pushed thewyardley/commented_out_code branch fromf50c9e4 tof451590CompareJune 15, 2024 16:54
@wyardley
Copy link
ContributorAuthor

@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.

Copy link
Contributor

@codejedi365codejedi365 left a 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
Copy link
Contributor

I agree it should operate on only the changed files

@codejedi365codejedi365 merged commit69af12c intopython-semantic-release:masterJun 15, 2024
@wyardleywyardley deleted the wyardley/commented_out_code branchJune 15, 2024 17:19
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@codejedi365codejedi365codejedi365 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wyardley@codejedi365

[8]ページ先頭

©2009-2025 Movatter.jp