-
Notifications
You must be signed in to change notification settings - Fork 237
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
Testing PR 7569 - Command Palette #1103
Conversation
I had assumed the old button would have been wrapped in a form, instead it does some JS magic to do the post instead.
Heya @basil! Total novice here when it comes to this sort of stuff, how would I update the PR to test https://ci.jenkins.io/job/Core/job/jenkins/job/PR-7569/? Thanks |
@janfaracik Wait until you get a passing CI build of your core PR that ends with this:
Then in your core PR go to the Checks tab, Jenkins Incrementals Publisher, Incrementals, View more details on Jenkins Incrementals Publisher. Note the incremental version and then update this PR with e.g. diff --git a/pom.xml b/pom.xml
index f5304c69..054efc7e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -31,7 +31,8 @@
<changelist>999999-SNAPSHOT</changelist>
<spotbugs.skip>true</spotbugs.skip>
- <jenkins.version>2.399</jenkins.version>
+ <!-- TODO https://github.com/jenkinsci/jenkins/pull/7569 -->
+ <jenkins.version>2.400-rc33526.0fb_9f1f080df</jenkins.version>
<selenium.version>4.8.1</selenium.version>
<guava.version>31.1-jre</guava.version> <!-- aligned with selenium -->
<aether.version>1.1.0</aether.version> |
converted to draft as this is not somethign we want to merge - and the test suite runs the same under a draft as non draft. |
Tests jenkinsci/jenkins#7569