Skip to content

Pin Node.js version #4148

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

Merged
merged 4 commits into from
Nov 8, 2021
Merged

Pin Node.js version #4148

merged 4 commits into from
Nov 8, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Nov 8, 2021

We currently use one Node.js version in production, another on CI and yet another one for our Docker setup. This PR pins them all to the same release (latest v14.x) and configures renovatebot to automatically keep the versions updated.

@Turbo87 Turbo87 requested review from JohnTitor and locks November 8, 2021 16:37
@Turbo87 Turbo87 added A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Nov 8, 2021
@locks
Copy link
Contributor

locks commented Nov 8, 2021

Pretty good 💪 Shouldn't CI pick up the version from package.json?

@locks
Copy link
Contributor

locks commented Nov 8, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Nov 8, 2021

📌 Commit 6184c96 has been approved by locks

@bors
Copy link
Contributor

bors commented Nov 8, 2021

⌛ Testing commit 6184c96 with merge 32d93b4...

@Turbo87
Copy link
Member Author

Turbo87 commented Nov 8, 2021

Shouldn't CI pick up the version from package.json?

not that I'm aware of 🤔

@locks
Copy link
Contributor

locks commented Nov 8, 2021

I may be confusing it with the volta GitHub Action 🤔

@bors
Copy link
Contributor

bors commented Nov 8, 2021

☀️ Test successful - checks-actions
Approved by: locks
Pushing 32d93b4 to master...

@bors bors merged commit 32d93b4 into rust-lang:master Nov 8, 2021
@Turbo87 Turbo87 deleted the node branch November 9, 2021 13:32
bors added a commit that referenced this pull request Nov 9, 2021
renovate: Adjust `node` package grouping

#4148 introduced a new way for renovatebot to update our Node.js version, but it wasn't perfectly compatible with the others yet. This PR fixes the issue, and unifies how renovatebot updates the Node.js versions in all of the relevant files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants