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
build: bump @electron/get to 2.0.0, node to >= 12+ #35301
Conversation
🤔 shouldn't the notes state that the minimum version of Node.js needed to install Electron itself is now 12.0.0? |
@malept Yeah, you’re right - updated release notes with the exact version of 12 👍 |
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.
Nit: I think I read somewhere that the correct way to refer to the runtime is with the .js
suffix.
-Notes: Bumps the minimum required version of Node needed to install Electron to 12.20.55
+Notes: Bumps the minimum required version of Node.js needed to install Electron to 12.20.55
Otherwise
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.
lgtm
Release Notes Persisted
|
* infra: bump @electron/get to 2.0.0, node to >= 12+ * chore: empty commit to trigger CircleCI
* infra: bump @electron/get to 2.0.0, node to >= 12+ * chore: empty commit to trigger CircleCI
Description of Change
Aligns our npm package with the newest version of @electron/get, and the new minimum version of Node that is required (see @electron/get PR here: electron/get#225 )
Checklist
npm test
passesRelease Notes
Notes: Bumps the minimum required version of Node.js needed to install Electron to 12.20.55