tests: update repo location for core deps by parthea · Pull Request #2566 · 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: update repo location for core deps#2566

Merged
parthea merged 3 commits into
mainfrom
update-repo-location
Feb 19, 2026
Merged

tests: update repo location for core deps#2566
parthea merged 3 commits into
mainfrom
update-repo-location

Conversation

@product-auto-label product-auto-label Bot added the size: s Pull request size is small. label Feb 19, 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

The pull request correctly updates the repository locations for google-api-core and proto-plus in the noxfile configurations by modifying the gapic/templates/noxfile.py.j2 template. However, several noxfile.py files within the tests/integration/goldens directories, which appear to be generated, have also been directly modified. According to the repository's general rules, fixes for generated code should be applied solely to the generator templates to ensure consistency across the codebase. These generated files should be updated by regenerating them after the template modification, rather than through direct edits.

Comment thread tests/integration/goldens/asset/noxfile.py
Comment thread tests/integration/goldens/asset/noxfile.py
Comment thread tests/integration/goldens/credentials/noxfile.py
Comment thread tests/integration/goldens/credentials/noxfile.py
Comment thread tests/integration/goldens/eventarc/noxfile.py
Comment thread tests/integration/goldens/logging_internal/noxfile.py
Comment thread tests/integration/goldens/redis/noxfile.py
Comment thread tests/integration/goldens/redis/noxfile.py
Comment thread tests/integration/goldens/redis_selective/noxfile.py
Comment thread tests/integration/goldens/redis_selective/noxfile.py
@parthea parthea marked this pull request as ready for review February 19, 2026 18:31
@parthea parthea requested a review from a team as a code owner February 19, 2026 18:31
@parthea parthea merged commit d15f5fa into main Feb 19, 2026
140 checks passed
@parthea parthea deleted the update-repo-location branch February 19, 2026 19:17
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