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

Quickstart update for native images #547

Merged
merged 14 commits into from
Apr 15, 2019
Merged

Quickstart update for native images #547

merged 14 commits into from
Apr 15, 2019

Conversation

romain-grecourt
Copy link
Contributor

@romain-grecourt romain-grecourt commented Apr 4, 2019

Make the helidon-quickstart-se example "GraalVM ready" instead of having a separate example.
Use a custom Maven plugin to execute the native-image command.

OSSRH does not have a hosted release repo available, thus a profile for ossrh-release does not make sense.
Move the nexus-staging-maven-plugin declaration with extension=true in the release profile.
…d-tools, bump the value from 1.0.9 to 1.0.10
@tomas-langer tomas-langer self-requested a review April 4, 2019 16:15
@romain-grecourt romain-grecourt self-assigned this Apr 4, 2019
@romain-grecourt

This comment has been minimized.

 - Use a shell script instead of archetype:create-from-project
 - Make all Dockerfiles multi-stage docker builds using Maven.
 - Move Dockerfiles and app.yaml at the top level.
@romain-grecourt romain-grecourt changed the title Helidon 546 Quickstart update for native images Apr 12, 2019
Copy link
Member

@tomas-langer tomas-langer left a comment

Choose a reason for hiding this comment

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

First part of review

@romain-grecourt romain-grecourt dismissed tomas-langer’s stale review April 15, 2019 23:15

Feedback has been incorporated, merging now to accomodate timezone difference

@romain-grecourt romain-grecourt merged commit ea23d46 into master Apr 15, 2019
@romain-grecourt romain-grecourt deleted the helidon-546 branch April 15, 2019 23:35
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