Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#704: Removing ArtifactResolver #828

Merged

Conversation

andrzejj0
Copy link
Contributor

Second stage: Removing ArtifactResolver.

@slawekjaranowski please review

@andrzejj0 andrzejj0 force-pushed the remove-artifact-resolver branch from 4dde10c to 2ec383d Compare November 24, 2022 19:10
@andrzejj0
Copy link
Contributor Author

Also managed to reduce the settings property (it's already available via MavenSession).

@andrzejj0 andrzejj0 force-pushed the remove-artifact-resolver branch from 2ec383d to 0a1cb95 Compare November 24, 2022 21:28
@andrzejj0
Copy link
Contributor Author

Changes applied.

@andrzejj0 andrzejj0 force-pushed the remove-artifact-resolver branch from 0a1cb95 to ffc4fb9 Compare November 25, 2022 20:26
@andrzejj0
Copy link
Contributor Author

@slawekjaranowski rebased and resolved conflicts

@slawekjaranowski slawekjaranowski added this to the 2.14.0 milestone Nov 29, 2022
@slawekjaranowski slawekjaranowski merged commit d420a62 into mojohaus:master Nov 29, 2022
@andrzejj0 andrzejj0 deleted the remove-artifact-resolver branch November 30, 2022 07:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants