Skip to content

Add ruff to pre-commit tooling & address findings - #820

Merged
freakboy3742 merged 3 commits into
beeware:mainfrom
AlexVerrico:AVerrico/ruff
Aug 31, 2026
Merged

freakboy3742 merged 3 commits into
beeware:mainfrom
AlexVerrico:AVerrico/ruff

Conversation

@AlexVerrico

@AlexVerrico AlexVerrico commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Add ruff and resolve findings Fixes #751
A number of long lines have not been wrapped, and have instead used # noqa to disable rust for those specific lines. This decision was made since this code is not customer-facing, and was generally less readable when wrapped.

PR Checklist:

  • I will abide by the BeeWare Code of Conduct
  • I have read and have followed the CONTRIBUTING.md file
  • This PR was generated or assisted using an AI tool

@freakboy3742 freakboy3742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great - thanks for the PR! I've pushed a couple of minor tweaks, mostly to narrow the scope of the noqa to just the E501 line length issue (otherwise, the noqa would mask any problem with the line.

@freakboy3742
freakboy3742 merged commit 123373c into beeware:main Aug 31, 2026
4 checks passed
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.

Add ruff to repo to lint Python scripts.

2 participants