8000 Update ruff, pyright, pre-commit, add-trailing-comma by pre-commit-ci[bot] · Pull Request #17671 · nvaccess/nvda · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update ruff, pyright, pre-commit, add-trailing-comma #17671

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jun 3, 2025
Merged

Update ruff, pyright, pre-commit, add-trailing-comma #17671

merged 4 commits into from
Jun 3, 2025

Conversation

pre-commit-ci[bot]
Copy link
Contributor
@pre-commit-ci pre-commit-ci bot commented Feb 3, 2025 •
edited by seanbudd
Loading

updates for linting:

  • ruff from 0.8.1 to 0.11.12
  • pre-commit from 4.0.1 to 4.2.0
  • pyright from 1.1.396 to 1.1.401

updates for pre-commit:

Merge as merge commit, to preserve the reference in .git-blame-ignore-revs

@pre-commit-ci pre-commit-ci bot requested a review from a team as a code owner February 3, 2025 23:20
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from dde09d7 to cc2bbd3 Compare February 3, 2025 23:20
@pre-commit-ci pre-commit-ci bot requested a review from seanbudd February 3, 2025 23:20
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Feb 3, 2025
@AppVeyorBot
Copy link

See test results for failed build of commit fe02c544cb

@seanbudd
Copy link
Member
seanbudd commented Feb 4, 2025 •
edited
Loading

Looks like we need to do another big automated lint - new Ruff rules!

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 616a43a to 22bf6d6 Compare March 3, 2025 20:12
@AppVeyorBot
Copy link

See test results for failed build of commit 7f5eba6422

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 739e9e9 to c17fda3 Compare April 7, 2025 20:11
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 4df9213 to 3756a80 Compare May 5, 2025 19:52
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 274a245 to 6383adb Compare June 2, 2025 19:53
@seanbudd
Copy link
Member
seanbudd commented Jun 3, 2025

I really don't like the way ruff auto formats multi-line strings. I've opened this issue astral-sh/ruff#18436 as an example

@seanbudd seanbudd force-pushed the pre-commit-ci-update-config branch from f75c657 to 1d74418 Compare June 3, 2025 05:39
@seanbudd seanbudd requested a review from a team as a code owner June 3, 2025 05:39
@seanbudd seanbudd requested a review from Qchristensen June 3, 2025 05:39
@seanbudd seanbudd marked this pull request as draft June 3, 2025 05:40
seanbudd
seanbudd reviewed Jun 3, 2025
View reviewed changes
@seanbudd seanbudd changed the title Pre-commit auto-update Update ruff, pyright, pre-commit, add-trailing-comma Jun 3, 2025
@seanbudd
update .git-blame-ignore-revs
4bf7317 
add string concat reference
@seanbudd seanbudd marked this pull request as ready for review June 3, 2025 06:03
@seanbudd seanbudd merged commit 85f0b96 into master Jun 3, 2025
14 of 15 checks passed
@seanbudd seanbudd deleted the pre-commit-ci-update-config branch June 3, 2025 06:26
@github-actions github-actions bot added this to the 2025.2 milestone Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seanbudd seanbudd seanbudd approved these changes

@Qchristensen Qchristensen Awaiting requested review from Qchristensen Qchristensen is a code owner automatically assigned from nvaccess/userdocs

Assignees
No one assigned
Labels
conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.
Projects
None yet
Milestone
2025.2
Development

Successfully merging this pull request may close these issues.
2 participants
@AppVeyorBot @seanbudd
0