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

remove lib.form.hetero-list.hetero-list #415

Merged
merged 3 commits into from
Nov 8, 2023

Conversation

car-roll
Copy link
Contributor

@car-roll car-roll commented Nov 6, 2023

See jenkinsci/jenkins#8418 (comment)

hetero-list adjunct was removed in jenkinsci/jenkins#8418
If this adjunct gets called, a large stack trace error should appear in the logs.

Testing done

adjunct resides in branchSourceSelector.jelly, which is called by configure-branches.jelly, then configure-entries.jelly from MultibranchProject. Tested under a local instance using a github multibranch folder but have not been able to extract any errors and was able to generate an error. This PR + jenkinsci/scm-api-plugin#228 removes the error

Submitter checklist

@car-roll car-roll added the bug label Nov 6, 2023
@car-roll car-roll requested a review from a team as a code owner November 6, 2023 20:00
@car-roll car-roll marked this pull request as draft November 6, 2023 20:04
@car-roll car-roll changed the title remove adjunct declaration and use new style button remove adjunct declaration Nov 6, 2023
@car-roll car-roll removed the bug label Nov 6, 2023
@car-roll car-roll marked this pull request as ready for review November 6, 2023 23:16
@car-roll car-roll added the chore label Nov 7, 2023
@@ -68,7 +68,7 @@
<revision>2</revision>
<changelist>999999-SNAPSHOT</changelist>
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
<jenkins.version>2.387.3</jenkins.version>
<jenkins.version>2.426</jenkins.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

change was introduced in 2.422, chose the (soon-to-be) next LTS

@car-roll car-roll changed the title remove adjunct declaration remove lib.form.hetero-list.hetero-list Nov 8, 2023
@car-roll car-roll added bug and removed chore labels Nov 8, 2023
@car-roll car-roll merged commit 8de8e78 into jenkinsci:master Nov 8, 2023
14 checks passed
@car-roll car-roll deleted the remove-adjunct branch November 8, 2023 18:41
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