Skip to content

chore(ndb): add missing format session and modernize lint and blacken sessions#17342

Merged
chalmerlowe merged 3 commits into
mainfrom
chore/add-nox-session-google-cloud-ndb
Jun 2, 2026
Merged

chore(ndb): add missing format session and modernize lint and blacken sessions#17342
chalmerlowe merged 3 commits into
mainfrom
chore/add-nox-session-google-cloud-ndb

Conversation

@chalmerlowe
Copy link
Copy Markdown
Contributor

@chalmerlowe chalmerlowe commented Jun 2, 2026

This PR adds missing format nox session and modernizes lint and blacken sessions to use Ruff for this package.

Additional changes:

  • Also marks blacken as deprecated.
  • Runs Ruff formatter and thus incorporates linting changes.

Changes to each nox session are based on the versions found in the gapic-generator noxfile.py.j2 template.

Fixes #17049

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 modernizes the codebase's formatting and import sorting by integrating Ruff into the development workflow, updating the linting sessions in noxfile.py, and cleaning up imports across numerous files. Feedback on these changes highlights two issues: first, the flake8 linting execution was accidentally omitted from the updated lint session in noxfile.py, leaving only formatting checks active; second, a typo ('functionother') in an error message within query.py should be corrected by adding a missing space.

Comment thread packages/google-cloud-ndb/noxfile.py
Comment thread packages/google-cloud-ndb/google/cloud/ndb/query.py Outdated
@chalmerlowe chalmerlowe force-pushed the chore/add-nox-session-google-cloud-ndb branch from 222bf2d to 56c04b2 Compare June 2, 2026 12:56
@chalmerlowe chalmerlowe force-pushed the chore/add-nox-session-google-cloud-ndb branch from 56c04b2 to aab9958 Compare June 2, 2026 13:02
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@chalmerlowe chalmerlowe added this to the Update nox sessions milestone Jun 2, 2026
@chalmerlowe chalmerlowe marked this pull request as ready for review June 2, 2026 17:27
@chalmerlowe chalmerlowe requested a review from a team as a code owner June 2, 2026 17:27
@chalmerlowe chalmerlowe merged commit 7f988ff into main Jun 2, 2026
32 checks passed
@chalmerlowe chalmerlowe deleted the chore/add-nox-session-google-cloud-ndb branch June 2, 2026 17:27
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.

[monorepo] Add or update nox sessions for packages/google-cloud-ndb/

2 participants