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

chore(deps): update all dependencies #3929

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions ai-platform/snippets/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"license": "Apache-2.0",
"author": "Google LLC",
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"files": [
"*.js"
Expand All @@ -25,4 +25,3 @@
"sinon": "^18.0.0"
}
}

4 changes: 2 additions & 2 deletions appengine/analytics/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Updating to Node.js 22.x is a good move, but ensure thorough testing of your application with this new version.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"start": "node app.js",
Expand All @@ -19,7 +19,7 @@
},
"dependencies": {
"express": "^4.16.4",
"node-fetch": "^3.0.0"
"node-fetch": "^2.7.0"
},
"devDependencies": {
"c8": "^10.0.0",
Expand Down
2 changes: 1 addition & 1 deletion appengine/building-an-app/build/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Updating to Node.js 22.x is recommended. Test your application thoroughly after this change.

Suggested change
"node": "22.x"
"node": "22.x"

},
"author": "Google Inc.",
"license": "Apache-2.0",
Expand Down
2 changes: 1 addition & 1 deletion appengine/building-an-app/update/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Good to see the Node.js version updated. Ensure all tests pass after this change.

Suggested change
"node": "22.x"
"node": "22.x"

},
"author": "Google Inc.",
"license": "Apache-2.0",
Expand Down
2 changes: 1 addition & 1 deletion appengine/datastore/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Node.js 22.x update looks good. Ensure compatibility and test thoroughly.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"start": "node app.js",
Expand Down
2 changes: 1 addition & 1 deletion appengine/endpoints/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Updating Node.js version is a good practice. Ensure your application works as expected with the new version.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"deploy": "gcloud app deploy",
Expand Down
2 changes: 1 addition & 1 deletion appengine/hello-world/flexible/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node app.js",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "=v16.20.2"
"node": "22.x"
},
"scripts": {
"start": "node app.js",
Expand Down
2 changes: 1 addition & 1 deletion appengine/hello-world/standard/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node app.js",
Expand Down
2 changes: 1 addition & 1 deletion appengine/memcached/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"license": "Apache Version 2.0",
"author": "Google Inc.",
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Updating Node.js to 22.x is a good step. Ensure compatibility and test your application thoroughly.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"start": "node app.js",
Expand Down
4 changes: 2 additions & 2 deletions appengine/metadata/flexible/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,15 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">= 18.x.x"
"node": "22.x"
},
"scripts": {
"system-test": "c8 mocha -p -j 2 --exit test/*.test.js",
"test": "npm run system-test"
},
"dependencies": {
"express": "^4.17.1",
"node-fetch": "^3.0.0"
"node-fetch": "^2.7.0"
},
"devDependencies": {
"c8": "^10.0.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,15 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "16.x.x"
"node": "22.x"
},
"scripts": {
"system-test": "c8 mocha -p -j 2 --exit test/*.test.js",
"test": "npm run system-test"
},
"dependencies": {
"express": "^4.17.1",
"node-fetch": "^2.6.1"
"node-fetch": "^2.7.0"
},
"devDependencies": {
"c8": "^8.0.0",
Expand Down
4 changes: 2 additions & 2 deletions appengine/metadata/standard/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,15 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Node.js version update looks good. Test your application after the update.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"system-test": "c8 mocha -p -j 2 --exit test/*.test.js",
"test": "npm run system-test"
},
"dependencies": {
"express": "^4.17.1",
"node-fetch": "^3.0.0"
"node-fetch": "^2.7.0"
},
"devDependencies": {
"c8": "^10.0.0",
Expand Down
2 changes: 1 addition & 1 deletion appengine/pubsub/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"license": "Apache Version 2.0",
"author": "Google Inc.",
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Updating the Node.js version is a good practice. Test your application after this change.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"start": "node app.js",
Expand Down
2 changes: 1 addition & 1 deletion appengine/static-files/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"license": "Apache Version 2.0",
"author": "Google Inc.",
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Node.js 22.x update looks good. Ensure your application is compatible and test thoroughly.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"start": "node app.js",
Expand Down
4 changes: 2 additions & 2 deletions appengine/storage/flexible/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"test": "c8 mocha -p -j 2 system-test/*.test.js --exit --timeout=30000"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"dependencies": {
"@google-cloud/storage": "^7.0.0",
Expand All @@ -15,7 +15,7 @@
"pug": "^3.0.2"
},
"devDependencies": {
"@types/express": "^4.17.17",
"@types/express": "^5.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Upgrading @types/express to v5 is a breaking change. Carefully review the v5 changelog (https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/express/CHANGELOG.md) and make the necessary code changes to ensure compatibility. Test thoroughly after updating.

Suggested change
"@types/express": "^5.0.0",
"@types/express": "^5.0.0"

"@types/multer": "^1.4.7",
"@types/proxyquire": "^1.3.28",
"@types/supertest": "^6.0.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"test": "c8 mocha -p -j 2 system-test/*.test.js --exit --timeout=30000"
},
"engines": {
"node": "16.x.x"
"node": "22.x"
},
"dependencies": {
"@google-cloud/storage": "^7.0.0",
Expand Down
4 changes: 2 additions & 2 deletions appengine/storage/standard/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"test": "c8 mocha -p -j 2 system-test/*.test.js --exit --timeout=30000"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"dependencies": {
"@google-cloud/storage": "^7.0.0",
Expand All @@ -15,7 +15,7 @@
"pug": "^3.0.2"
},
"devDependencies": {
"@types/express": "^4.17.17",
"@types/express": "^5.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Upgrading @types/express to v5 introduces breaking changes. Ensure your code is compatible and test thoroughly. Review the changelog for details.

Suggested change
"@types/express": "^5.0.0",
"@types/express": "^5.0.0"

"@types/multer": "^1.4.7",
"@types/proxyquire": "^1.3.28",
"@types/supertest": "^6.0.0",
Expand Down
8 changes: 4 additions & 4 deletions appengine/typescript/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"license": "Apache Version 2.0",
"author": "Google Inc.",
"engines": {
"node": "20.x"
"node": "22.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low

Node.js version update looks good. Test your application after the update.

Suggested change
"node": "22.x"
"node": "22.x"

},
"scripts": {
"prepare": "npm run gcp-build",
Expand All @@ -26,11 +26,11 @@
"express": "^4.16.3"
},
"devDependencies": {
"@types/express": "^4.17.17",
"@types/node": "^20.0.0",
"@types/express": "^5.0.0",
"@types/node": "^22.0.0",
"c8": "^10.0.0",
"chai": "^4.5.0",
"gts": "^5.0.0",
"gts": "^6.0.0",
Comment on lines +29 to +33
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Upgrading these dev dependencies is good. However, ensure that your tests and build process still work correctly with the new versions, especially with the breaking change in @types/express v5. Check its changelog for details.

Suggested change
"@types/express": "^5.0.0",
"@types/node": "^22.0.0",
"c8": "^10.0.0",
"chai": "^4.5.0",
"gts": "^5.0.0",
"gts": "^6.0.0",
"@types/express": "^5.0.0",
"@types/node": "^22.0.0",
"c8": "^10.0.0",
"chai": "^4.5.0",
"gts": "^6.0.0"

"mocha": "^10.2.0",
"typescript": "^5.0.0",
"wait-port": "^1.0.0"
Expand Down
2 changes: 1 addition & 1 deletion appengine/websockets/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"license": "Apache Version 2.0",
"author": "Google Inc.",
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"deploy": "gcloud app deploy",
Expand Down
4 changes: 2 additions & 2 deletions asset/snippets/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"license": "Apache-2.0",
"author": "Google LLC",
"engines": {
"node": ">=12.0.0"
"node": "22.x"
},
"files": [
"*.js"
Expand All @@ -25,7 +25,7 @@
"devDependencies": {
"c8": "^10.0.0",
"chai": "^4.5.0",
"gts": "^5.0.0",
"gts": "^6.0.0",
"mocha": "^10.0.0",
"sinon": "^18.0.0",
"uuid": "^10.0.0"
Expand Down
2 changes: 1 addition & 1 deletion auth/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"test:auth": "c8 mocha -p -j 2 system-test/auth.test.js --timeout=30000",
Expand Down
2 changes: 1 addition & 1 deletion automl/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"license": "Apache-2.0",
"author": "Google LLC",
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"repository": "googleapis/nodejs-automl",
"private": true,
Expand Down
2 changes: 1 addition & 1 deletion batch/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"license": "Apache-2.0",
"author": "Google LLC",
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"files": [
"*.js"
Expand Down
2 changes: 1 addition & 1 deletion cloud-language/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"license": "Apache-2.0",
"author": "Google Inc.",
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"repository": "googleapis/nodejs-language",
"private": true,
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/mysql/mysql/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

# Use the official lightweight Node.js image.
# https://hub.docker.com/_/node
FROM node:20-slim
FROM node:22-slim

# Create and change to the app directory.
WORKDIR /app
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/mysql/mysql/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node server/server.js",
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/mysql/mysql2/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

# Use the official lightweight Node.js image.
# https://hub.docker.com/_/node
FROM node:20-slim
FROM node:22-slim
# Create and change to the app directory.
WORKDIR /app

Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/mysql/mysql2/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node server/server.js",
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/postgres/knex/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

# Use the official lightweight Node.js image.
# https://hub.docker.com/_/node
FROM node:20-slim
FROM node:22-slim
# Create and change to the app directory.
WORKDIR /app

Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/postgres/knex/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node server/server.js",
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/sqlserver/mssql/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# Use the official lightweight Node.js image.

# https://hub.docker.com/_/node
FROM node:20-slim
FROM node:22-slim
# Create and change to the app directory.
WORKDIR /app

Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/sqlserver/mssql/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node server/server.js",
Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/sqlserver/tedious/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
# Use the official lightweight Node.js image.

# https://hub.docker.com/_/node
FROM node:20-slim
FROM node:22-slim
# Create and change to the app directory.
WORKDIR /app

Expand Down
2 changes: 1 addition & 1 deletion cloud-sql/sqlserver/tedious/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"url": "https://github.com/GoogleCloudPlatform/nodejs-docs-samples.git"
},
"engines": {
"node": ">=16.0.0"
"node": "22.x"
},
"scripts": {
"start": "node server/server.js",
Expand Down
Loading
Loading