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

[MENFORCER-195] Fix to enforcer dependencyConvergence rule to support… #54

Closed
wants to merge 1 commit into from
Closed

[MENFORCER-195] Fix to enforcer dependencyConvergence rule to support… #54

wants to merge 1 commit into from

Conversation

fbyrne
Copy link

@fbyrne fbyrne commented Jul 16, 2019

Hi,
Adding this pull request as a fix for [MENFORCER-195].

Please disregard if the existing PR is sufficient.

Kind regards,
Fergus

@fbyrne
Copy link
Author

fbyrne commented Jul 16, 2019

Duplicate of #20

@fbyrne fbyrne closed this Jul 16, 2019
@fbyrne
Copy link
Author

fbyrne commented Jul 22, 2019

Reopening as PR #20 is open since 2016 with no activity.

@fbyrne fbyrne reopened this Jul 22, 2019
@rfscholte
Copy link
Contributor

I worked together with Ray Tsang on this, after a lot of digging we discovered the real fix requires a change in Maven Core, see apache/maven#269.
This PR doesn't have a test to confirm the fix, #56 does.
So I'll close this one in favor of the other two PRs

@rfscholte rfscholte closed this Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants