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.

Sign up for GitHub

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

Jump to bottom

[pom] Per maven standards, make certain artifacts provided #308

Merged
merged 1 commit into from
Mar 21, 2022
Merged

[pom] Per maven standards, make certain artifacts provided #308

merged 1 commit into from
Mar 21, 2022

Conversation

hazendaz
Copy link
Collaborator

  • added two that need to be provided that are compile in usage. This is now required upon backport to maven plugin plugin that throws warnings to the same. These specific libraries ship with maven and therefore are not needed to repeat. This keeps m2 repository cleaner.

@hazendaz
[pom] Per maven standards, make certain artifacts provided
d4ddf8f 
- added two that need to be provided that are compile in usage.  This is now required upon backport to maven plugin plugin that throws warnings to the same.  These specific libraries ship with maven and therefore are not needed to repeat.  This keeps m2 repository cleaner.
@hazendaz hazendaz requested a review from mathieucarbou March 21, 2022 01:06
@mathieucarbou mathieucarbou merged commit 6be109a into mathieucarbou:master Mar 21, 2022
@mathieucarbou mathieucarbou added this to the 4.2 milestone Mar 21, 2022
@mathieucarbou mathieucarbou added is:enhancement Enhancement to an existing feature in:build MLP build system (maven) labels Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mathieucarbou mathieucarbou mathieucarbou approved these changes

Assignees
No one assigned
Labels
in:build MLP build system (maven) is:enhancement Enhancement to an existing feature
Projects
None yet
Milestone
4.2
Development

Successfully merging this pull request may close these issues.
2 participants
@hazendaz @mathieucarbou