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

add support for npm 11 #10795

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add support for npm 11 #10795

wants to merge 1 commit into from

Conversation

LitoMore
Copy link
Contributor

@LitoMore LitoMore commented Jan 9, 2025

Changes

  • Add npm 11 to engines
  • Use v11 for Node.js 22
  • Update the Dockerfile npm version to v10 to keep the version consistent (in test-package-lib.yml it's v10)
  • Simply npm versions from ^9.0.0 || ^10.0.0 to ^9 || ^10. The new format is also consistent with the version in GitHub Actions files:
    matrix:
    include:
    - node: '16'
    npm: '^9'
    engine-strict: 'false'
    - node: '18'
    npm: '^9'
    engine-strict: 'false'
    - node: '20'
    npm: '^10'
    engine-strict: 'true'
    - node: '22'
    npm: '^10'

Copy link
Contributor

github-actions bot commented Jan 9, 2025

Messages
📖 ✨ Thanks for your contribution to Shields, @LitoMore!

Generated by 🚫 dangerJS against 5c5967f

@LitoMore LitoMore changed the title add suppport for npm 11 add support for npm 11 Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant