Skip to content

Conversation

@BobDu
Copy link
Member

@BobDu BobDu commented Aug 7, 2025

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@BobDu BobDu force-pushed the remove-commons-beanutils branch from 63d2780 to fa10b1f Compare August 7, 2025 05:55
Signed-off-by: Bob Du <[email protected]>
@basil
Copy link
Member

basil commented Aug 7, 2025

Isn't this just inlining it rather than removing it? Which means we have the maintenance version of maintaining the inlined copy. Plus we wouldn't be able to remove the BeanUtils JAR from Jenkins core either, because it's used by Stapler/Jelly.

@BobDu BobDu marked this pull request as draft August 8, 2025 03:42
@BobDu
Copy link
Member Author

BobDu commented Aug 8, 2025

I also discovered this issue during further exploration.

Let's wait and see if apache/commons-beanutils#359 is accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants