-
Notifications
You must be signed in to change notification settings - Fork 1
chore(deps): upgrade dependencies to latest version where possible #3288
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 updates various npm dependencies to their latest compatible versions, excluding packages with breaking changes that require code modifications (express v5, storybook v9, openid-client v6, @guardian/ophan-tracker-js 2.6.1, winston 3.18.1, and eslint-config updates).
Key changes:
- Updated development dependencies including testing tools (Cypress, Jest types), build tools (@swc/core, webpack preprocessor), and type definitions
- Updated AWS SDK packages from v3.922.0 to v3.930.0
- Updated package manager from pnpm 10.20.0 to 10.22.0
Reviewed Changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Updated 21 development and production dependencies to latest compatible versions |
| eslint.config.mjs | Removed tsconfig project configuration from ESLint parser options |
| cdk/package.json | Updated CDK-related dependencies and AWS CDK packages |
Files not reviewed (1)
- cdk/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| './tsconfig.json', | ||
| './cypress/tsconfig.json', | ||
| './scripts/okta/tsconfig.json', | ||
| ], |
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.
why was this change needed ?
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.
I got this error when bumping eslint error Parsing error: Enabling "project" does nothing when "projectService" is enabled. You can remove the "project" setting. Apparently the projectService is the newer version that doesn't need to be explicit as it'll apply the nearest tsconfig.json https://typescript-eslint.io/packages/parser/#projectservice
What does this change?
This PR updates various npm dependencies to their latest compatible versions, excluding packages with breaking changes that require code modifications (express v5, storybook v9, openid-client v6, @guardian/ophan-tracker-js 2.6.1, winston 3.18.1, and eslint-config updates).
In eslint, The
projectproperty has been deprecated (https://typescript-eslint.io/packages/parser/#project) andprojectService(https://typescript-eslint.io/packages/parser/#projectservice) will automatically use the nearesttsconfig.json.#3283