Skip to content

Conversation

@MarkEWaite
Copy link
Contributor

Test more versions of Jenkins core

  • Test with Jenkins 2.361.3 (Java 11) and 2.375 (Java 17)
  • Fix Jenkins 2.375 compile failure

Broader compilation checks with multiple Java versions and slightly
broader test coverage by checking multiple Jenkins versions.
@MarkEWaite MarkEWaite requested a review from a team as a code owner November 11, 2022 14:01
Collection<Trigger> triggerList = p.getTriggers().values();
for (Trigger t : triggerList) {
ParameterizedJobMixIn.ParameterizedJob<?, ?> p = (ParameterizedJobMixIn.ParameterizedJob) job;
Collection<Trigger<?>> triggerList = p.getTriggers().values();
Copy link
Member

Choose a reason for hiding this comment

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

(subsuming #1358)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that I will merge #1358 once it completes its CI run so that the changelog includes the text from #1358. I merged it into this pull request to assure that this pull request would also verify.

Copy link
Member

Choose a reason for hiding this comment

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

FYI, topologically (not squash- or rebase-) merging this PR would cause #1358 to be considered merged as well, I think according to Release Drafter too.

Copy link
Member

Choose a reason for hiding this comment

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

Basically, a PR is “merged” when either

  • it was squash- or rebase-merged by GitHub; or
  • its head commit becomes an ancestor of its base branch, by any means (including e.g. git merge without any GH-specific tools).

@MarkEWaite MarkEWaite changed the title Test versions of Jenkins core Test more versions of Jenkins core Nov 11, 2022
@MarkEWaite MarkEWaite enabled auto-merge (rebase) November 11, 2022 15:03
@MarkEWaite
Copy link
Contributor Author

@gounthar or @jmMeessen could you approve this pull request? Sorry to bother you on a holiday, but it is easier if another maintainer approves the pull request. If no approval is received, I'll override and merge it myself without the extra approval.

@MarkEWaite MarkEWaite disabled auto-merge November 11, 2022 15:34
@MarkEWaite MarkEWaite merged commit ebd63be into jenkinsci:master Nov 11, 2022
@MarkEWaite MarkEWaite deleted the test-more-versions branch November 11, 2022 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants