Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to latest google cloud storage php #88

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Yugloocamai
Copy link

This gets us away from a versioning issue that Google introduced today. GoogleCloudPlatform/google-cloud-php-storage versions less than v1.7.1 no longer work. See googleapis/google-cloud-php-core@6f1b06c

Attempt to circumvent problem with Google Cloud dependencies by bumping the version.
Update composer.json: versioning fix
@Yugloocamai
Copy link
Author

Yugloocamai commented Sep 13, 2018

We can optionally wait on this PR, it looks like the upstream repo is being looked at. See latest comment. googleapis/google-cloud-php-core@6f1b06c

@Yugloocamai
Copy link
Author

Yugloocamai commented Sep 13, 2018

Google fixed the issue, but we may want to merge anyway? Stay up to date?
googleapis/google-cloud-php#1292

@nicja
Copy link
Contributor

nicja commented Sep 20, 2018

It looks like between the google fix and the update in ver 7.1.0 this is no longer an issue, you can use whichever 1.* series cloud storage library now?

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.

2 participants