chore(cli): Java init template compiles apps before trying to run them

[ghost]

The TypeScript and C# init templates use commands that compile the code before running it to prevent the user from forgetting. This adds the same functionality to the Java init template.

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[ghost]

@RomainMuller Can you approve this, assuming I've done it the best way? I'd like to get it into a release soon; it'll let me remove lots of build instructions from the Dev Guide.

[RomainMuller]

I reckon this can be achieved without shelling out twice (tested locally and achieved the expected result).

[RomainMuller]

I'd like to get it into a release soon; it'll let me remove lots of build instructions from the Dev Guide.

Yeah that definitely is a sweet, sweet simplification! Thanks for looking into that!!

[ghost]

@RomainMuller Ready for another look.

[ghost]

I reckon this can be achieved without shelling out twice (tested locally and achieved the expected result).

I thought about trying it in one command but was afraid it might still try to run the app if the compile failed. On reflection, even it did that, you're just going to get the same result you already had, so no harm. And avoiding spinning up the JVM twice seems like a definite win.

[shivlaks]

@jerry-aws nice! this is a great simplification.

were you able to run the integ tests successfully as well? yarn integ-init or npm run integ-init
since our PR build doesn't run these, might be worth running it for sanity ahead of the merge.

[RomainMuller]

Annotated with do-not-merge so you can consider @shivlaks' comment. You can remove the label when you're ready to merge in.

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[ghost]

I was able to run the integ tests successfully after a bit of effort.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
• Commit ID: a9c08b8
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).