-
Notifications
You must be signed in to change notification settings - Fork 21
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
add java docker container #89
Conversation
1da0b91
to
bd26da0
Compare
docker/pulumi/Dockerfile
Outdated
# Install Java | ||
RUN apt-get update -y && \ | ||
apt-get install -y \ | ||
maven |
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.
For readability, suggest either indenting this b/c it's a param of the previous line, or merging into the previous line.
docker/java/Dockerfile
Outdated
|
||
RUN apt-get update -y && \ | ||
apt-get install -y \ | ||
java-common \ |
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.
Same suggestion: either indent or put all on one line.
d630cb4
to
9cac395
Compare
Adds a java docker image.
Fixes #88
Fixes pulumi/pulumi-java#529