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

[JENKINS-71715] Update minimum supported Jenkins version to 2.440 to be able to use new ConsoleUrlProvider API to link to Pipeline build console page #373

Merged
merged 7 commits into from
Jan 29, 2024

Conversation

dwnusbaum
Copy link
Member

@dwnusbaum dwnusbaum commented Aug 8, 2023

See JENKINS-71715 and jenkinsci/jenkins#8321.

Testing done

Tested in combination with jenkinsci/pipeline-input-step-plugin#145 and jenkinsci/pipeline-graph-view-plugin#289, see this video:

Screen.Recording.2024-01-29.at.3.03.32.PM.mov

Submitter checklist

pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
@timja
Copy link
Member

timja commented Jan 20, 2024

ready to go @dwnusbaum?

@dwnusbaum
Copy link
Member Author

@timja Yeah I was just waiting for the new LTS to drop; I'll try to update all of the related PRs this week.

@dwnusbaum dwnusbaum changed the title [JENKINS-71715] Use new ConsoleUrlProvider API to link to Pipeline build console page [JENKINS-71715] Update minimum supported Jenkins version to 2.440 to be able to use new ConsoleUrlProvider API to link to Pipeline build console page Jan 29, 2024
@dwnusbaum dwnusbaum changed the title [JENKINS-71715] Update minimum supported Jenkins version to 2.440 to be able to use new ConsoleUrlProvider API to link to Pipeline build console page [JENKINS-71715] Update minimum supported Jenkins version to 2.440 to be able to use new ConsoleUrlProvider API to link to Pipeline build console page Jan 29, 2024
@dwnusbaum dwnusbaum marked this pull request as ready for review January 29, 2024 19:49
@dwnusbaum dwnusbaum requested a review from a team as a code owner January 29, 2024 19:49
Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

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

Sanity tested recently?

@dwnusbaum
Copy link
Member Author

Sanity tested recently?

Yes, I just added a new video to the PR description.

@jglick jglick merged commit 7fd111b into jenkinsci:master Jan 29, 2024
14 checks passed
@dwnusbaum dwnusbaum deleted the JENKINS-71715 branch January 29, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants