Skip to content

Conversation

@matthewmayer
Copy link
Contributor

Adds a more precise engines field

https://nodejs.org/api/modules.html#loading-ecmascript-modules-using-require

This ensures, for example, that a developer using Node 22.0.0 will receive a warning if they attempt to install v10 so they can bump their version to v22.13+

@matthewmayer matthewmayer requested a review from a team as a code owner July 6, 2025 09:10
@netlify
Copy link

netlify bot commented Jul 6, 2025

Deploy Preview for fakerjs ready!

Name Link
🔨 Latest commit 1cee233
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/686a3e3cdb87cd0008d4e207
😎 Deploy Preview https://deploy-preview-3561.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Jul 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.96%. Comparing base (5d74cfd) to head (1cee233).
Report is 1 commits behind head on next.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #3561      +/-   ##
==========================================
- Coverage   99.96%   99.96%   -0.01%     
==========================================
  Files        2888     2888              
  Lines      221958   221958              
  Branches      931      929       -2     
==========================================
- Hits       221891   221889       -2     
- Misses         67       69       +2     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xDivisionByZerox xDivisionByZerox added p: 1-normal Nothing urgent c: infra Changes to our infrastructure or project setup labels Jul 6, 2025
@xDivisionByZerox xDivisionByZerox added this to the v10.0 milestone Jul 6, 2025
@xDivisionByZerox xDivisionByZerox added this pull request to the merge queue Jul 6, 2025
Merged via the queue into next with commit 8972060 Jul 6, 2025
25 checks passed
@xDivisionByZerox xDivisionByZerox deleted the more-precise-node-versions branch July 6, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: infra Changes to our infrastructure or project setup p: 1-normal Nothing urgent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants