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

[MNG-7909] Apply Resolver 2.0.0 changes #317

Merged
merged 5 commits into from
Nov 7, 2023
Merged

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Nov 7, 2023

Changes:

  • IT MNG-4368 "smart install" undone
  • IT MNG-5175 force Wagon as it uses Wagon config
  • IT MNG-7470 adapt to new default and new transport

Changes:
* IT MNG-4368 "smart install" undone
* IT MNG-5175 force Wagon as it uses Wagon config
* IT MNG-7470 adapt to new default and new transport
@cstamas cstamas requested a review from gnodet November 7, 2023 10:35
@cstamas cstamas self-assigned this Nov 7, 2023
@gnodet
Copy link
Contributor

gnodet commented Nov 7, 2023

You need to make your changes conditional to the maven version I suppose.

@cstamas
Copy link
Member Author

cstamas commented Nov 7, 2023

I had all the assumptions inverted :) This should make it, also applied some cleanup

@gnodet
Copy link
Contributor

gnodet commented Nov 7, 2023

I had all the assumptions inverted :) This should make it, also applied some cleanup

Yeah, much better now. The assumptions are much easier to read and the test actually works.

@cstamas cstamas merged commit c7c8923 into apache:master Nov 7, 2023
3 checks passed
@cstamas cstamas deleted the MNG-7909 branch November 7, 2023 12:27
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