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

Platform JSON: support reading BOM's managed deps from a generated POM #11650

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

aloubyansky
Copy link
Member

This change supports picking up the content of the POM generated on the fly for the current project that has replaced the original project's POM.

@boring-cyborg boring-cyborg bot added the area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins label Aug 26, 2020
@aloubyansky aloubyansky force-pushed the generate-json-from-pom branch 3 times, most recently from 673c4f0 to 70e8ec1 Compare August 28, 2020 06:25
@aloubyansky aloubyansky added this to the 1.8.0 - master milestone Aug 28, 2020
@gsmet gsmet force-pushed the generate-json-from-pom branch from 70e8ec1 to 6c48842 Compare August 30, 2020 16:32
@gsmet
Copy link
Member

gsmet commented Aug 30, 2020

Force pushed a rebase.

@gsmet
Copy link
Member

gsmet commented Aug 31, 2020

CI failure is unrelated. @quarkusio/devtools could one of you have a look? I have no idea what this PR does.

@aloubyansky
Copy link
Member Author

@gsmet this change enables generating the BOM for the platform and use it for running the testsuite. Without it, the original BOM would be used in place of the generated one.

@gsmet gsmet removed this from the 1.8.0.CR1 milestone Sep 2, 2020
@gsmet
Copy link
Member

gsmet commented Sep 2, 2020

@aloubyansky I'm not entirely sure this is needed for 1.8? I think we will need time to test and validate the platform changes and I'm not sure it's something we can do in time for 1.8. WDYT?

@aloubyansky
Copy link
Member Author

I would like to have it in a released Quarkus version for my platform tests. The SNAPSHOT versions are not always passing the platform testsuite due to some incompatibilities.

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

OK, thanks for the explanation. Will get it in 1.8.0.

@gsmet gsmet merged commit 8b6cdae into quarkusio:master Sep 3, 2020
@gsmet gsmet modified the milestones: 1.9.0 - master, 1.8.0.Final Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants