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
[WFLY-12802] Quickstart for MP Config 1.4 #375
Conversation
@emmartins I believe this is ready for review. Only OpenShift tests are missing which are not a part of this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
-DarchetypeGroupId=org.apache.maven.archetypes \ | ||
-DarchetypeArtifactId=maven-archetype-webapp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just wondering, why not use the wildfly archetype?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rhusar is there any without Java/Jakarta mentions? I think that is why I chose to stick with default maven ones.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we could use any WildFly archetypes we should have merged Stefano's archetype - the problem is none of them are presently available to us once we use the quickstarts with EAP.
* Simple tests for MicroProfile Config quickstart. Arquillian deploys an @AR archive to the application server, which | ||
* contains several endpoints exposing injected configuration values and verifies that they are correctly invoked. | ||
* | ||
* @author <a href="mstefank@redhat.com>Martin Stefanko</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing ending "
import java.nio.file.Paths; | ||
|
||
/** | ||
* Simple tests for MicroProfile Config quickstart. Arquillian deploys an @AR archive to the application server, which |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/@AR/WAR
.addAsResource("META-INF/services/org.eclipse.microprofile.config.spi.ConfigSourceProvider") | ||
.addAsResource("META-INF/services/org.eclipse.microprofile.config.spi.Converter") | ||
// enable CDI | ||
.addAsManifestResource(EmptyAsset.INSTANCE, "beans.xml"); } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} should be on a new line.
7982e43
to
7ecf6be
Compare
@xstefank Needs a rebase now |
https://issues.redhat.com/browse/WFLY-12802