arturobernalg commented on PR #591: URL: https://github.com/apache/jspwiki/pull/591#issuecomment-6768226928
@juanpablo-santos the red Jenkins check on this PR is not caused by the change, as far as I can tell from the log of PR-591 build #1. Jenkins takes the Jenkinsfile from this PR (5da981c86), but each of the three parallel branches (JDK 11, 17 and 21) then runs `git url: buildRepo` and checks out refs/remotes/origin/master (1ee556bbf), not the PR. master asks for JDK 17 in the enforcer, so the JDK 11 build stops with "MUST USE JDK 17" and the pipeline ends in FAILURE, while the other branches built the whole reactor. This PR only touches ChangeLog.md, Release.java, PropertyReader and its test. So Jenkins is not testing the code of this PR, and I would expect the same red result for other PRs against master-2.x. The workflows of GitHub Actions only exist on master, so there is no other CI here. A fix would be in the Jenkinsfile of master-2.x (check out the PR with `checkout scm`, and do not build with JDK 11 what asks for 17). I have not touched it. I am not sure which of the parallel branches printed the error, because the logs are interleaved. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
