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

fix build container to have later npm #1570

Merged
merged 1 commit into from Jul 30, 2019
Merged

fix build container to have later npm #1570

merged 1 commit into from Jul 30, 2019

Conversation

bkendall
Copy link
Contributor

Description

The default npm in the node:8.16.0 container changes the package-lock.json file which breaks deploying. Update the version to something a hair later, and everything's okay (it's what just released firebase-tools!).

@googlebot googlebot added the cla: yes Manual indication that this has passed CLA. label Jul 30, 2019
@coveralls
Copy link

Coverage Status

Coverage remained the same at 63.942% when pulling 9656e1d on bk-fix-build into 91eabfb on master.

@bkendall bkendall merged commit 74ea036 into master Jul 30, 2019
@bkendall bkendall deleted the bk-fix-build branch July 30, 2019 22:47
joehan added a commit that referenced this pull request Nov 13, 2019
* fix build container to have later npm (#1570)

* Release Firestore emulator v1.7.0 (#1571)

* Use ADC for Emulator Tests (#1575)

* Adds privacy policy link to login notice. (#1576)

* Use displayName when available (#1555)

* shows displayName if available

* formats

* uses displayName if available

* formats

* Declare project and app metadata types (#1470) and update dependencies

* Implement projects:create command (#1429)

* Implement api calls to support project creation

* Initial working command

* Replace nock with sinon to mock api requests

* export createFirebaseProject() and update tests

* Implement projects:create command

* Remove poll helper methods

* Rename firebase-resource-manager.ts to projectsCreate.ts

* Declare project and app metadata types

* Update timeout constant

* Update unit tests as suggestion

* Split metadata.ts into apps.ts and projects.ts

* move code to projects.ts

* Addressing comment

* Implement apps:create command (#1438)

* Implement API calls for apps:create command

* Implement apps:create command

* Remove --sha-certificate option

* Support providing appStoreId for iOS app creation

* Update the command to use getProjectId()

* Update unit tests as suggestion

* move code to apps.ts

* Add input options interfaces and getAppPlatform

* Implement projects:list command (#1465)

* Implement projects:list command

* Matching changes from metadata

* Move code to projects.ts

* Resolving TODO

* Implement apps:list command (#1477)

* Implement apps:list command

* Introducing AppPlatform.ANY and code refactoring

* Move code to apps.ts

* Implement getFirebaseProject and deprecate firebaseApi (#1502)

* Re-implement `firebase init` command's default project setup flow (#1506)

* Implement getFirebaseProject and deprecate firebaseApi

* Implement getProjectPage() and refactor listFirebaseProject()

* Implement use an existing project option

* Update behavior when there's no project

* Implement "Create a new project" option (#1510)

* [projects:list] and [apps:list] Logging count (#1526)

* Logging total project count after listing.

* Logging total app count

* Implement apps:sdkconfig command (#1515)

* Implement API methods for apps:sdkconfig

* Implement apps:sdkconfig command

* Add getOrPromptProject function to projects.ts (#1552)

* Make getAppPlatform throw if the input platform is unknown

* Update changelog for project and app management

* Move changelog into CHANGELOG.MD

* No more requireAuth() for functions (#1577)

* Add the ability to add Firebase resources to an existing GCP project (#1572)

* Implement projects:addfirebase command

* Update init command to have add Firebase option

* implements style fixes suggested in other pr

* typescriptifys ensureApiEnabled (#1554)

* typescriptifys ensureApiEnabled

This reverts commit e7ec80f.

* style fixes for switch to ts

* Release Firestore emulator v1.7.1 (#1584)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Manual indication that this has passed CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants