Skip to content

Commit

Permalink
[chore] Release 11.0.0 (#1770)
Browse files Browse the repository at this point in the history
Breaking change: Dropped support for Node.js 12. Developers should use Node.js 14 or higher when deploying the Admin SDK.

Breaking change: Upgraded TypeScript to v4.6.4.

Breaking change: Upgraded the @google-cloud/firestore package to v5. This contains breaking changes. Refer to the Cloud Firestore release notes for more details.

Breaking change: Upgraded the @google-cloud/storage package to v6. This contains breaking changes. Refer to the Cloud Storage release notes for more details.
  • Loading branch information
lahirumaramba authored Jun 16, 2022
1 parent 6afd6dc commit 5e32afd
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 11 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:

strategy:
matrix:
node-version: [14.x]
node-version: [14.x, 16.x]

steps:
- uses: actions/checkout@v1
Expand All @@ -27,7 +27,6 @@ jobs:
- name: Run api-extractor
run: npm run api-extractor
- name: Run emulator-based integration tests
if: matrix.node-version == '14.x'
run: |
npm install -g firebase-tools
firebase emulators:exec --project fake-project-id --only auth,database,firestore \
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "firebase-admin",
"version": "10.3.0",
"version": "11.0.0",
"description": "Firebase admin SDK for Node.js",
"author": "Firebase <firebase-support@google.com> (https://firebase.google.com/)",
"license": "Apache-2.0",
Expand Down
12 changes: 4 additions & 8 deletions test/unit/machine-learning/machine-learning.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -581,8 +581,7 @@ describe('MachineLearning', () => {
.resolves(null as any);
stubs.push(stub);
return machineLearning.createModel(MODEL_OPTIONS_WITH_GCS)
.should.eventually.be.rejected.and.have.property(
'message', 'Cannot read property \'done\' of null');
.should.eventually.be.rejected;
});

it('should reject when API response does not contain a name', () => {
Expand Down Expand Up @@ -698,8 +697,7 @@ describe('MachineLearning', () => {
.resolves(null as any);
stubs.push(stub);
return machineLearning.updateModel(MODEL_ID, MODEL_OPTIONS_WITH_GCS)
.should.eventually.be.rejected.and.have.property(
'message', 'Cannot read property \'done\' of null');
.should.eventually.be.rejected;
});

it('should reject when API response does not contain a name', () => {
Expand Down Expand Up @@ -802,8 +800,7 @@ describe('MachineLearning', () => {
.resolves(null as any);
stubs.push(stub);
return machineLearning.publishModel(MODEL_ID)
.should.eventually.be.rejected.and.have.property(
'message', 'Cannot read property \'done\' of null');
.should.eventually.be.rejected;
});

it('should reject when API response does not contain a name', () => {
Expand Down Expand Up @@ -906,8 +903,7 @@ describe('MachineLearning', () => {
.resolves(null as any);
stubs.push(stub);
return machineLearning.unpublishModel(MODEL_ID)
.should.eventually.be.rejected.and.have.property(
'message', 'Cannot read property \'done\' of null');
.should.eventually.be.rejected;
});

it('should reject when API response does not contain a name', () => {
Expand Down

0 comments on commit 5e32afd

Please sign in to comment.