-
Notifications
You must be signed in to change notification settings - Fork 800
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 volta configuration for managing node versions #19705
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -146,5 +146,9 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"node": "14.16.0", | ||
"yarn": "1.22.10" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
Significance: minor | ||
Type: added | ||
|
||
Added volta configuration to package.json to pin node version |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,5 +35,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"extends": "../../../package.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
Significance: minor | ||
Type: added | ||
|
||
Added volta configuration to package.json to pin node version |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,5 +29,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't know if we'd want to extend here (and for the other packages), since the JITM package is also available standalone, in a situation where there is nothing to extend: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If nothing else, we could copy the configuration and use a script (run from precommit and CI) to ensure that the copies remain in sync and are in sync with .nvmrc. We could also verify that the "engines" are in sync everywhere too, and that the versions are compatible. Or we could wait for volta to resolve volta-cli/volta#282 so we could just drop a .voltarc or whatever in the monorepo root. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Makes sense! I'd forgotten about the mirror repos. I've updated the PR so that each package.json has the version info rather than extending the root, except for There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
FYI, I pushed #19748 to ensure that Once those are merged, it should be straightforward to do the same for the volta blocks. |
||
"extends": "../../../package.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
Significance: minor | ||
Type: added | ||
|
||
Added volta configuration to package.json to pin node version |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,5 +29,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"extends": "../../../package.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
Significance: minor | ||
Type: other | ||
|
||
Added volta configuration to package.json to pin node version |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -215,5 +215,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"extends": "../../../package.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -55,5 +55,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"extends": "../../../../../package.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -25,5 +25,8 @@ | |
"engines": { | ||
"node": "^14.16.0", | ||
"yarn": "^1.3.2" | ||
}, | ||
"volta": { | ||
"extends": "../../../../package.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.
Seems not really "minor". It doesn't change the code at all, it's just configuring a dev tool.
Same in your other changelog files.
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.
Updated.