Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

fix(deps): drop pkg_resources#744

Merged
parthea merged 1 commit into
maingoogleapis/python-storage:mainfrom
drop-dependency-on-pkg-resourcesgoogleapis/python-storage:drop-dependency-on-pkg-resourcesCopy head branch name to clipboard
Mar 30, 2022
Merged

fix(deps): drop pkg_resources#744
parthea merged 1 commit into
maingoogleapis/python-storage:mainfrom
drop-dependency-on-pkg-resourcesgoogleapis/python-storage:drop-dependency-on-pkg-resourcesCopy head branch name to clipboard

Conversation

@parthea

@parthea parthea commented Mar 30, 2022

Copy link
Copy Markdown
Contributor

Fixes #740 🦕

google-cloud-core>=1.6.0 is already listed as a dependency in setup.py. We should remove code that uses pkg_resources to check google-cloud-core>=1.6.0.

"google-cloud-core >= 1.6.0, < 3.0dev",

@parthea
parthea requested review from a team March 30, 2022 13:32
@product-auto-label product-auto-label Bot added the api: storage Issues related to the googleapis/python-storage API. label Mar 30, 2022

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

Thank you!

@parthea
parthea merged commit e963f33 into main Mar 30, 2022
@parthea
parthea deleted the drop-dependency-on-pkg-resources branch March 30, 2022 23:37
@andrewsg

andrewsg commented Apr 4, 2022

Copy link
Copy Markdown
Contributor

Thanks a lot, very helpful!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/python-storage API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add setuptools to dependencies

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.