Skip to content

chore(deps): bump firebase deps - #702

Merged
hiranya911 merged 1 commit into
firebase:masterfrom
arjunyel:chore/firebase-deps
Nov 12, 2019
Merged

chore(deps): bump firebase deps#702
hiranya911 merged 1 commit into
firebase:masterfrom
arjunyel:chore/firebase-deps

Conversation

@arjunyel

@arjunyelarjunyel commented Nov 12, 2019

Copy link
Copy Markdown
Contributor

Update Firebase dependencies, all tests passed locally

RELEASE NOTE: feat(Cloud Firestore): Upgraded @google-cloud/firestore dependency version to 2.6.0.
RELEASE NOTE: change(Cloud Storage): Upgraded @google-cloud/storage dependency version to 4.1.2. This version contains some minor breaking changes. Check release notes of the dependency for more information.

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

Storage 4.0 has some breaking changes, but their impact seems to be quite low. So this should be ok.

@hiranya911hiranya911 self-assigned this Nov 12, 2019
@hiranya911
hiranya911 merged commit ef231fb into firebase:masterNov 12, 2019
@arjunyel

Copy link
Copy Markdown
ContributorAuthor

Thanks for the merge! Is it possible to cut a new release or how does that work?

@hiranya911

hiranya911 commented Nov 13, 2019

Copy link
Copy Markdown
Contributor

We can try and get a release out next week. @lahirumaramba FYI.

@lahirumaramba

Copy link
Copy Markdown
Member

@arjunyel@hiranya911 this is now included in release v8.8.0

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@arjunyel@hiranya911@lahirumaramba