Skip to content

Conversation

@risantos
Copy link
Member

@risantos risantos commented Jun 4, 2025

Description

After update

❯ yarn audit
yarn audit v1.22.22
0 vulnerabilities found - Packages audited: 389
✨  Done in 0.59s.

- Replace `pre-commit` with `@fastify/[email protected]`
- Update `[email protected]`
- Run `yarn upgrade` to update all dependencies
Copilot AI review requested due to automatic review settings June 4, 2025 11:31
@risantos risantos added the dependencies Pull requests that update a dependency file label Jun 4, 2025
@risantos risantos self-assigned this Jun 4, 2025
Copy link

Copilot AI left a 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 development dependencies to address security vulnerabilities and ensure tooling is current.

  • Replaces the old pre-commit package with @fastify/pre-commit@^2.2.0
  • Bumps eslint-config-uphold to ^6.5.2 and adjusts the ESLint version specifier
  • Runs yarn upgrade to update all dependencies and clear audit warnings
Comments suppressed due to low confidence (2)

package.json:27

  • [nitpick] Dev dependencies are not sorted alphabetically. Consider ordering entries alphabetically to improve readability and consistency.
"@fastify/pre-commit": "^2.2.0",

package.json:29

  • [nitpick] Using a tilde (~) version range for ESLint restricts patch updates only; if you want to include minor updates, consider using a caret (^) for non-breaking version bumps.
"eslint": "~9.28.0",

Copy link
Member

@Kamefrede Kamefrede left a comment

Choose a reason for hiding this comment

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

LGTM

@Kamefrede Kamefrede merged commit 548e837 into master Jun 4, 2025
3 checks passed
@risantos risantos deleted the support/update-dependencies branch June 4, 2025 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants