Skip to content

Conversation

@crwilcox
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 17, 2019
@crwilcox crwilcox force-pushed the restrict-resumable-media branch from b129833 to 508c6e7 Compare September 17, 2019 23:18
"google-auth >= 1.2.0",
"google-cloud-core >= 1.0.3, < 2.0dev",
"google-resumable-media >= 0.4.0",
"google-resumable-media >= 0.3.1, < 0.5.0dev",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a match to the manually released 1.19.1
We can rev this to 0.4.0 since this was an unreleased change
@tseaver

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"google-resumable-media >= 0.3.1, < 0.5.0dev",
"google-resumable-media >= 0.4.0, < 0.5.0dev",

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ISTM that >= 0.3.1, != 0.4.0, < 0.5dev is the right specification, until GRMP has a new version (might be 1.0.0?) to include deliberately: the reality is that 0.3.1 and 0.4.1 are near identical, and the storage client works with the older one.

@tseaver tseaver changed the title Restrict version range of google-resumable-media Storage: Restrict version range of google-resumable-media. Sep 18, 2019
Copy link
Contributor

@tseaver tseaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR needs the test change reversion, already in #9240. I recommend closing here, and adding a PR which updates the changelog separately.

@tseaver
Copy link
Contributor

tseaver commented Sep 18, 2019

Superseded by #9240.

@tseaver tseaver closed this Sep 18, 2019
@tseaver tseaver deleted the restrict-resumable-media branch August 24, 2021 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants