tests: downgrade mypy to 1.16.0 by parthea · Pull Request #2580 · googleapis/gapic-generator-python · GitHub
Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

tests: downgrade mypy to 1.16.0#2580

Merged
parthea merged 1 commit into
mainfrom
downgrade-mypy
Mar 11, 2026
Merged

tests: downgrade mypy to 1.16.0#2580
parthea merged 1 commit into
mainfrom
downgrade-mypy

Conversation

@parthea

@parthea parthea commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Partial revert of #2530 due to downstream mypy failures: #2579

@parthea parthea requested a review from a team as a code owner March 11, 2026 14:03
@product-auto-label product-auto-label Bot added the size: s Pull request size is small. label Mar 11, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

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 downgrades mypy to <1.16.0 to address downstream failures, which is a reasonable temporary fix. The change is correctly applied to the noxfile.py.j2 template. However, the corresponding golden files under tests/integration/goldens/ have been manually patched instead of being regenerated from the updated template. My review includes a comment recommending that these files be regenerated to ensure consistency and adhere to the project's contribution guidelines.

Comment thread tests/integration/goldens/asset/noxfile.py
@parthea parthea merged commit 4f0c8ee into main Mar 11, 2026
113 checks passed
@parthea parthea deleted the downgrade-mypy branch March 11, 2026 16:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants