-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Keep node versions in sync #89
Conversation
Signed-off-by: Alexander Brandes <mc.cache@web.de>
Maybe replace dependabot all together by renovabot at the same time? Dependabot annoyingly doesn't update version in comments, ex: https://github.com/jenkins-infra/incrementals-publisher/pull/79/files |
For sure! |
Signed-off-by: Alexander Brandes <mc.cache@web.de>
Signed-off-by: Alexander Brandes <mc.cache@web.de>
@@ -1 +1 @@ | |||
nodejs 18.16.1 | |||
nodejs 21.6.1 |
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.
Use an old version to check renovate works maybe?
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.
https://github.com/NotMyFault/incrementals-publisher/issues/6
Applied this PR as patch NotMyFault@f9a1e7b and renovate detects it in its dashboard
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.
Actually, I removed the regex check after renovate picked it up as asdf over regex in the issue linked above.
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.
Is this possible to keep an LTS line to avoid bad surprises?
Nevermind, I see the 21 line is already used and we won't downgrade
Signed-off-by: Alexander Brandes <mc.cache@web.de>
I've requested an installation of renovate on this repo. |
The change proposed switches to renovate to ensure the node version used to build on the CI and used in the Dockerfile remains aligned, see #73 diverging between the node versions.
Additionally, I removed the unused .nvmrc file, given 14 can't be used anyway.