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

Run Ruff --fix & --no-qa #16752

Closed
wants to merge 2 commits into from
Closed

Run Ruff --fix & --no-qa #16752

wants to merge 2 commits into from

Conversation

seanbudd
Copy link
Member

@seanbudd seanbudd commented Jun 27, 2024

This is a sample PR for #16751

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 27, 2024 03:53
@seanbudd seanbudd requested review from SaschaCowley and removed request for a team June 27, 2024 03:53
@seanbudd seanbudd marked this pull request as draft June 27, 2024 03:53
@seanbudd seanbudd mentioned this pull request Jun 27, 2024
11 tasks
@seanbudd seanbudd changed the base branch from useRuff to master June 27, 2024 04:27
@seanbudd seanbudd closed this Jun 27, 2024
@seanbudd seanbudd reopened this Jun 27, 2024
@seanbudd seanbudd changed the base branch from master to useRuff June 27, 2024 04:27
@seanbudd
Copy link
Member Author

PR checks succeeded as expected, closing

@seanbudd seanbudd closed this Jun 27, 2024
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.

1 participant