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

Integrate maven-java into CI build #36

Merged

Conversation

octonato
Copy link
Member

No description provided.

@octonato octonato changed the base branch from main to rgc/feature-branch-maven-java June 15, 2021 19:38
@octonato octonato force-pushed the rgc/include-maven-plugin-in-ci branch from adcf2a5 to e210a0e Compare June 16, 2021 12:00
@octonato octonato force-pushed the rgc/include-maven-plugin-in-ci branch from 40def82 to b303374 Compare June 16, 2021 19:35
@octonato octonato marked this pull request as ready for review June 16, 2021 19:52
Comment on lines +181 to +183
mvn -B archetype:generate -DgroupId=com.example -DartifactId=counter-event-sourced-entity -DarchetypeGroupId=com.lightbend -DarchetypeArtifactId=maven-archetype-akkasls-event-sourced-entity -DarchetypeVersion=$SDK_VERSION
cd counter-event-sourced-entity
mvn -B -Dakkaserverless-sdk.version=$SDK_VERSION compile
Copy link
Member Author

Choose a reason for hiding this comment

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

I'm taking a different approach here than the original build GH Action. The original build was deploying the proxy, the user service and then hitting it with curl.

I confess, that the main reason that I'm not reproducing it here is because it's a little more complicated to do it in CircleCI. I could have tried harder, but at this stage I'm taking a shortcut. I prefer to move faster with the merge of the projects.

Also, I believe we will be better served if we use the integration test testkit with a fully implement application and this is something I would like to add once we move the codegen under the SDK project.

Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

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

LGTM.

@octonato octonato merged commit effd62e into rgc/feature-branch-maven-java Jun 17, 2021
@octonato octonato deleted the rgc/include-maven-plugin-in-ci branch June 17, 2021 07:27
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.

None yet

2 participants