Skip to content
This repository was archived by the owner on Mar 15, 2025. It is now read-only.

fix(deps): require python 3.6 #46

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Jun 20, 2021

google-cloud-access-context-manager is a proto only package that is only used by google-cloud-asset. The last version of google-cloud-asset that supports python 2.7 is 1.3.0. See the setup.py of version 1.3.0 here. In version 1.3.0 of python-asset, the version of google-cloud-access-context-manager that is required has an upper bound of < 0.2.0dev. Assuming that the SemVer minor version of google-cloud-access-context-manager increases after this PR, this change should not break users that are still on python-asset version 1.3.0.

@parthea parthea requested a review from a team as a code owner June 20, 2021 14:22
@product-auto-label product-auto-label bot added the api: accesscontextmanager Issues related to the googleapis/python-access-context-manager API. label Jun 20, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 20, 2021
@parthea
Copy link
Contributor Author

parthea commented Jun 22, 2021

Wait for #47

@parthea parthea added do not merge Indicates a pull request not ready for merge, due to either quality or timing. and removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Jun 22, 2021
@parthea
Copy link
Contributor Author

parthea commented Jun 22, 2021

@busunkim96 This is ready for review

@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Jun 22, 2021
@parthea
Copy link
Contributor Author

parthea commented Jun 22, 2021

Thanks @busunkim96 !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: accesscontextmanager Issues related to the googleapis/python-access-context-manager API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants