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

[JENKINS-65161] remove commons-digester2 from core and upgrade plugins to commons-digester3 #15

Merged
merged 1 commit into from
Jun 3, 2021

Conversation

olamy
Copy link
Member

@olamy olamy commented Apr 30, 2021

No description provided.

…ns to commons-digester3

Signed-off-by: olivier lamy <olamy@apache.org>
@olamy
Copy link
Member Author

olamy commented Apr 30, 2021

see jenkinsci/jenkins#5320

@rsandell
Copy link
Member

rsandell commented May 3, 2021

I reran the build to make it succeed as it looked like an environmental issue.

@alecharp
Copy link
Member

alecharp commented May 4, 2021

@jlamasrios do you think this PR could be merged and you could cut a release of the plugin with it? Thanks.

@batmat batmat requested a review from jlamasrios May 4, 2021 20:03
@batmat
Copy link
Member

batmat commented May 26, 2021

Hi @jlamasrios, FYI we are about to merge the Core PR that relates to this change.

Without merging & releasing with this current PR, this plugin will be broken in the first weekly shipped with the core change.

Thanks!

@jlamasrios
Copy link
Contributor

Hi, thanks for the PR and sincere apologies for the delay!

I'll look into it to merge it asap.

@jlamasrios jlamasrios merged commit 0fc8c56 into jenkinsci:master Jun 3, 2021
@jlamasrios
Copy link
Contributor

jlamasrios commented Jun 3, 2021 via email

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.

6 participants