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

Fixes the extension author guide WRT the new quarkus-extension.yaml #5805

Merged
merged 1 commit into from
Nov 27, 2019

Conversation

loicmathieu
Copy link
Contributor

No description provided.

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 for catching that!

I added a comment inline.

@@ -378,7 +378,7 @@ The above sequence of commands does the following:
A Maven build performed immediately after generating the modules should fail due to a `fail()` assertion in one of the test classes.

There is one step (specific to the Quarkus source tree) that you should do manually when creating a new extension:
Add the extension coordinates to `devtools/common/src/main/filtered/extensions.json`.
Create a `quarkus-extension.yaml` file that describe your extension inside the runtime module `src/main/resources/META-INF` folder.
Copy link
Member

Choose a reason for hiding this comment

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

So the error was here before but Create should be create after a colon. Can you fix that while you're at it?

I also think an example of an existing extension that has all the right information would be helpful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, I add an example from Agroal

@gsmet gsmet merged commit e266e81 into quarkusio:master Nov 27, 2019
@gsmet
Copy link
Member

gsmet commented Nov 27, 2019

Thanks!

@gsmet gsmet added this to the 1.0.1.Final milestone Nov 27, 2019
@loicmathieu loicmathieu deleted the fix/extension-author-guide branch December 3, 2019 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants