Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

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

Merged
merged 3 commits into from
May 24, 2021

Conversation

olamy
Copy link
Member

@olamy olamy commented Apr 19, 2021

No description provided.

…s to commons-digester3

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

olamy commented Apr 19, 2021

see jenkinsci/jenkins#5320

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

olamy commented Apr 29, 2021

@mig42 ping as you are the main maintainer

@mig42
Copy link
Contributor

mig42 commented Apr 29, 2021

Sure, we'll get this merged and released ASAP 😊

pom.xml Outdated Show resolved Hide resolved
rsandell
rsandell previously approved these changes May 3, 2021
import hudson.scm.ChangeLogParser;
import hudson.scm.ChangeLogSet;
import hudson.scm.RepositoryBrowser;
import org.apache.commons.digester3.Digester;

Choose a reason for hiding this comment

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

The important line

@alecharp
Copy link
Member

alecharp commented May 4, 2021

@mig42 any blockers to get this PR merge and cut a release of the plugin? Thanks.

@batmat batmat requested a review from mig42 May 4, 2021 20:24
@olamy
Copy link
Member Author

olamy commented May 24, 2021

@mig42 ping :)

Copy link
Contributor

@mig42 mig42 left a comment

Choose a reason for hiding this comment

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

Looks good to me! I'll merge the PR once I verified it works in my environment.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants