FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore(deps): bump firebase deps by arjunyel · Pull Request #702 · firebase/firebase-admin-node · GitHub

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 commented Nov 12, 2019
edited by hiranya911
Loading

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.

hiranya911 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM.

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

hiranya911 self-assigned this Nov 12, 2019
hiranya911 merged commit ef231fb into firebase:master Nov 12, 2019

Copy link
Copy Markdown
Contributor Author

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

hiranya911 commented Nov 13, 2019
edited
Loading

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL