Skip to content
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

Dropping 2.332.x & 2.346.x #1668

Merged
merged 2 commits into from Jan 4, 2023
Merged

Dropping 2.332.x & 2.346.x #1668

merged 2 commits into from Jan 4, 2023

Conversation

jglick
Copy link
Member

@jglick jglick commented Jan 3, 2023

#1667 (comment) and in many other cases plugins will be 2.361.x+, so it is cumbersome to update them past that switch (jenkinsci/plugin-pom#478), and older BOM releases will become increasingly quiescent anyway. Will probably also unblock #1647 though I am not attempting that here.

@jglick jglick added the breaking label Jan 3, 2023
@jglick jglick requested review from basil and timja January 3, 2023 22:23
@jglick jglick added the full-test Test all LTS lines in this PR and do not halt upon first error. label Jan 3, 2023
@@ -7,9 +7,6 @@ updates:
- "jglick"
schedule:
interval: "daily"
ignore:
# TODO until 2.332.x & 2.346.x dropped
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was obsolete as of #1642 anyway.

@@ -7,8 +7,7 @@ assert artifactMap['junit:junit'] == project.artifactMap['junit:junit']
def managedPluginDeps = managedDeps.collect {stripAllButGA(it)}.grep { ga ->
def art = artifactMap[ga]
if (art == null) {
if (ga.contains('.plugins') // TODO without an Artifact, we have no reliable way of checking whether it is actually a plugin
&& !(ga == 'io.jenkins.plugins:ionicons-api' && settings.activeProfiles.any {it ==~ /^2[.](332)[.]x$/})) { // TODO: Remove once 2.332.x is no longer part of the BOM (or if MNG-5600 is fixed and we can exclude this dependency in the BOM for old LTS lines)
if (ga.contains('.plugins')) { // TODO without an Artifact, we have no reliable way of checking whether it is actually a plugin
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jglick jglick requested a review from MarkEWaite January 3, 2023 22:26
@jglick jglick enabled auto-merge (squash) January 3, 2023 22:42
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very good to me

@jglick jglick merged commit 0bf2af7 into jenkinsci:master Jan 4, 2023
@jglick jglick deleted the java11 branch January 4, 2023 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking full-test Test all LTS lines in this PR and do not halt upon first error.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants