-
Notifications
You must be signed in to change notification settings - Fork 196
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
Bump plugin from 4.48 to 4.49 #603
Conversation
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.
Tried interactively and the basics of the Pipeline editor seem to be working.
Rephrased https://github.com/jenkinsci/plugin-pom/releases/tag/plugin-4.49, to highlight what users need to look for when specifying a version. |
@@ -69,6 +69,7 @@ | |||
<groovy-cps.version>1.34</groovy-cps.version> | |||
<node.version>16.17.0</node.version> | |||
<npm.version>8.18.0</npm.version> | |||
<yarn.version>1.22.19</yarn.version> |
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.
Not listed on https://github.com/yarnpkg/berry/releases; is this just the previous hard-coded value or something?
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.
This is the last release of the 1.x line. 1.x is unsupported for several years, but the current supported line, berry (3.x), requires jenkinsci/plugin-pom#625 on our end first, before working out of the box.
Picking up #601 (comment)
@jglick 4.49 removed the default the versions set for yarn, node and npm. This repository uses all three tools, but did declare
<node.version>
and<npm.version>
only.This PR demonstrates how to upgrade to 4.49 by specifying
<yarn.version>
.I'll add a few links to the 4.49 changelog where to obtain these versions from. I'd assume that makes it easier for users to upgrade?
Closes #601