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

Depend on commons-lang3-api plugin #445

Merged
merged 1 commit into from
May 31, 2022
Merged

Conversation

sghill
Copy link
Contributor

@sghill sghill commented May 30, 2022

Hi, I noticed there is a commons-lang3-api plugin now. I thought jira-plugin would be a good candidate for adopting commons-lang3-api, since other plugins that may also need commons-lang3 rely on it (ex: artifactory), and similar api plugins (ex: jackson2-api) have already been adopted.

See jenkinsci/jenkins#6267 (comment) for details on plugin introduction.

🚨 Please review the guidelines for contributing to this repository.

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side) and not your master branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub (n/a)
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue (existing tests work)

@rantoniuk rantoniuk requested a review from olamy May 30, 2022 16:48
@olamy olamy added the dependencies Release Drafter label label May 31, 2022
@olamy
Copy link
Member

olamy commented May 31, 2022

@sghill thanks for your contribution

@olamy olamy merged commit f8cf619 into jenkinsci:master May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Release Drafter label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants