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

Document create-extension Maven mojo #3390

Merged
merged 3 commits into from
Aug 2, 2019
Merged

Document create-extension Maven mojo #3390

merged 3 commits into from
Aug 2, 2019

Conversation

ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Jul 31, 2019

No description provided.

* Java keywords elimination in the generated package name
* Remove -runtime suffix in the runtime dependency artifactId in
Deployment pom.xml
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.

Thanks @ppalaga!

I added a few comments/suggestions/questions inline.

docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/extension-authors-guide.adoc Outdated Show resolved Hide resolved
@gsmet
Copy link
Member

gsmet commented Jul 31, 2019

BTW, the build is failing.

Copy link
Contributor Author

@ppalaga ppalaga left a comment

Choose a reason for hiding this comment

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

All requests should be solved in b4067d0. Please re-review.

@ppalaga
Copy link
Contributor Author

ppalaga commented Aug 1, 2019

All review comments should be resolved in 18d4bff.

@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Aug 1, 2019
@ppalaga
Copy link
Contributor Author

ppalaga commented Aug 2, 2019

The CI passed. Could you please merge, @gsmet ? I'd like to send a followup that makes the mojo add the newly created artifacts to the BOMs.

@gsmet gsmet merged commit 4917d2c into quarkusio:master Aug 2, 2019
@geoand geoand added this to the 0.21.0 milestone Aug 8, 2019
@gsmet gsmet changed the title Fix #3386 Document create-extension Maven mojo Document create-extension Maven mojo Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants