Skip to content

chore(nox): chore update noxfile with format session that uses ruff#16647

Draft
chalmerlowe wants to merge 4 commits intomainfrom
chore-update-to-ruff
Draft

chore(nox): chore update noxfile with format session that uses ruff#16647
chalmerlowe wants to merge 4 commits intomainfrom
chore-update-to-ruff

Conversation

@chalmerlowe
Copy link
Copy Markdown
Contributor

@chalmerlowe chalmerlowe commented Apr 14, 2026

Partial fix for #16014

Provides the following update to multiple noxfiles:

  • Adds a version number for ruff
  • Adds OR edits a format nox session to use ruff for formatting
  • Formats the edited noxfiles.

What it does NOT do:

This is not a universal panacea... it is purposefully kept lean and focuses on a single task.
It does not delete black versions, it does not delete isort versions, etc.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request replaces isort and black with ruff for code formatting and import sorting across multiple packages. The review identifies several issues, including missing tool installations, missing session implementations, and the use of undefined variables. It also recommends removing redundant formatting sessions, restoring type hints, and refactoring duplicated path lists to improve maintainability.

chalmerlowe and others added 2 commits April 14, 2026 09:08
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@chalmerlowe
Copy link
Copy Markdown
Contributor Author

Unit test failures are in sqlalchemy-spanner.
No changes introduced in this PR should cause the unit tests to fail. I believe this is an unrelated issue that is out of scope for this update.

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