Skip to content

style(ruff): enable pyupgrade (UP) rules and fix them all #889

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

noirbizarre
Copy link
Member

Description

Enable pyupgrade rules in ruff to automatically update syntax according to supported Python version.
Then apply it on all the repository.

Checklist

  • Add test cases to all the changes you introduce (N/A)
  • Run ./scripts/format and ./scripts/test locally to ensure this change passes linter check and test
  • Test the changes on the local machine manually
  • Update the documentation for the changes (N/A)

Expected behavior

No changes

Additional context

@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Files Coverage Δ
commitizen/__version__.py 100.00% <100.00%> (ø)
commitizen/bump.py 100.00% <100.00%> (ø)
commitizen/changelog.py 100.00% <100.00%> (+0.50%) ⬆️
commitizen/changelog_parser.py 97.01% <100.00%> (ø)
commitizen/cli.py 94.20% <ø> (ø)
commitizen/commands/bump.py 97.66% <100.00%> (+0.01%) ⬆️
commitizen/commands/changelog.py 98.93% <100.00%> (ø)
commitizen/commands/check.py 100.00% <100.00%> (ø)
commitizen/commands/commit.py 98.64% <100.00%> (+0.05%) ⬆️
commitizen/config/json_config.py 100.00% <100.00%> (+6.45%) ⬆️
... and 17 more

📢 Thoughts on this report? Let us know!.

@Lee-W Lee-W added pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check and removed pr-status: wait-for-review labels Oct 17, 2023
@Lee-W Lee-W merged commit a48d263 into commitizen-tools:master Oct 17, 2023
@noirbizarre noirbizarre deleted the style/pyupgrade branch October 17, 2023 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-status: ready-to-merge almost ready to merge. just keep it for a few days for others to check scope: pre-commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants