Skip to content

fix: use requirements files with hashes instead of constraints#17512

Open
parthea wants to merge 9 commits into
mainfrom
requirements-prototype-2026-06-09
Open

fix: use requirements files with hashes instead of constraints#17512
parthea wants to merge 9 commits into
mainfrom
requirements-prototype-2026-06-09

Conversation

@parthea

@parthea parthea commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Towards b/510816107

Fixes #17531
Closes #17532

This PR transitions dependency management for testing from constraint files to hashed requirements.

To update hashed requirements, run the following in the root of packages/gapic-generator

export PYENV_VERSION=3.10:3.11:3.12:3.13:3.14:3.15
nox -s update_hashes

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

The number of changes in this pull request is too large for Gemini Code Assist to generate a review.

@parthea parthea force-pushed the requirements-prototype-2026-06-09 branch 3 times, most recently from fb6ff68 to 39dcbaa Compare June 18, 2026 18:04
@parthea parthea self-assigned this Jun 22, 2026
@parthea parthea force-pushed the requirements-prototype-2026-06-09 branch 7 times, most recently from 07b15e9 to d4b4eb1 Compare June 23, 2026 16:02
@parthea parthea force-pushed the requirements-prototype-2026-06-09 branch from d4b4eb1 to d039a24 Compare June 23, 2026 16:48
@parthea parthea marked this pull request as ready for review June 23, 2026 17:29
@parthea parthea requested a review from a team as a code owner June 23, 2026 17:29
@parthea parthea assigned chalmerlowe and ohmayr and unassigned parthea Jun 23, 2026

@hebaalazzeh hebaalazzeh 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.

LGTM

@hebaalazzeh hebaalazzeh 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.

LGTM

@chalmerlowe chalmerlowe 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.

LGTM

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.

gapic-generator: use install_unittest_dependencies in prerelease_deps and core_deps_from_source

4 participants