-
-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
[FIXED JENKINS-46082] API will include culprits again. #2978
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐝 In longer term we also need a fix in Stapler
This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation. |
@oleg-nenashev Agreed, but this didn't seem like worth waiting. |
I agree. If nobody votes against, will merge it into the next weekly to get soaked for the next x.1 LTS |
Is this expected to be resolved in Stapler any time soon? AFAIUI that's a Java 8 specific limitation of that...? |
@abayer could you please create a ticket to Stapler? |
@oleg-nenashev What component etc do I use? |
@abayer https://github.com/stapler/stapler/issues + Mirror issue to |
@reviewbybees done @jenkinsci/code-reviewers I kindly ask for the expedited review since it is a regression. |
See JENKINS-46082.
Proposed changelog entries
Submitter checklist
* Use the
Internal:
prefix if the change has no user-visible impact (API, test frameworks, etc.)Desired reviewers
@reviewbybees