Skip to content
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

Lint the repository with Ruff #16766

Merged
merged 3 commits into from
Jun 28, 2024
Merged

Lint the repository with Ruff #16766

merged 3 commits into from
Jun 28, 2024

Conversation

seanbudd
Copy link
Member

This is a follow up for #16751
Must be merge commit not squash merge

Using #16751, the following commands were applied:

  1. ruff check --fix to apply automatic fixes
  2. ruff check --add-noqa to add no-qa comments to all lint failures, so that issues that can't be automatically fixed don't trigger future failures

@coderabbitai ignore

@seanbudd seanbudd requested a review from a team as a code owner June 28, 2024 01:42
@seanbudd seanbudd requested a review from SaschaCowley June 28, 2024 01:42
@seanbudd seanbudd mentioned this pull request Jun 28, 2024
11 tasks
@AppVeyorBot
Copy link

See test results for failed build of commit b8456ffa02

@seanbudd seanbudd merged commit 405f9bf into master Jun 28, 2024
2 of 3 checks passed
@seanbudd seanbudd deleted the applyRuff2 branch June 28, 2024 02:48
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.

2 participants