-
Notifications
You must be signed in to change notification settings - Fork 1
chore: upgraded to node 22 #19
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR upgrades the project from Node 20 (LTS Iron) to Node 22, including significant dependency updates and code adjustments to align with the new Node version and updated libraries.
- Upgraded all AWS SDK packages from v3.645.0 to v3.929.0
- Upgraded Middy middleware from v4.7.0 to v6.4.5 (major version bump)
- Updated development tooling including Nx (19.6.5 → 22.0.3), TypeScript (5.5.4 → 5.9.3), and ESLint dependencies
- Removed callback parameter from middyfy handler invocation to align with Middy v6 changes
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| .nvmrc | Updated Node.js LTS version reference from "lts/iron" to "lts/jod" |
| .github/workflows/npm-publish.yml | Updated Node.js version from 20.x to 22.x in CI workflow |
| .github/workflows/checks.yml | Updated Node.js version from 20.x to 22.x in CI workflow |
| package.json | Upgraded AWS SDK packages, Middy middleware (v4→v6), Nx tooling (v19→v22), TypeScript, and ESLint dependencies; added @middy/util package |
| packages/api/package.json | Bumped package version from 0.1.0 to 0.2.4 |
| packages/api/src/lib/apiHandler/apiHandler.ts | Removed callback parameter from middyfy handler invocation (Middy v6 compatibility) |
| packages/s3/tsconfig.lib.json | Added "node" to types array for Node.js type definitions |
| packages/dynamo/tsconfig.lib.json | Added "node" to types array for Node.js type definitions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
|



No description provided.