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

[4.x]: Change provided and optional to just optional - iteration 2 (#6495) #6650

Merged
merged 4 commits into from
May 5, 2023

Conversation

Captain1653
Copy link
Contributor

Part of #6495

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Apr 19, 2023
Copy link
Contributor

@romain-grecourt romain-grecourt left a comment

Choose a reason for hiding this comment

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

Is it possible to group all the pom changes required to fix #6495 in this PR ?

@Captain1653
Copy link
Contributor Author

Yes, it's possible. I changed only 'helidon-config-metadata-processor' and I'm not sure about the others. A full list of modules would be extremely useful.

@romain-grecourt
Copy link
Contributor

romain-grecourt commented Apr 19, 2023

You will have to do a text search to find all pom files that have both <scope>provided</scope> and <optional>true</optional>. The duplication is most likely to occur when <optional>true</optional> is present.

@Captain1653
Copy link
Contributor Author

That's all. I didn't find any <scope>provided</scope> and <optional>true</optional> dependency.

@Captain1653
Copy link
Contributor Author

@romain-grecourt Approve it again, please. I fixed conflicts in nima/webclient/webclient/pom.xml

@barchetta barchetta merged commit 82916f5 into helidon-io:main May 5, 2023
@Captain1653 Captain1653 deleted the 6495-provided-vs-optional-2 branch May 6, 2023 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants