Skip to content

Conversation

@jbern0rd
Copy link
Contributor

@jbern0rd jbern0rd commented Feb 8, 2022

No description provided.

@jbern0rd jbern0rd requested a review from a team February 8, 2022 17:47
Copy link
Contributor

@james-toussaint james-toussaint left a comment

Choose a reason for hiding this comment

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

Good!

@@ -1,23 +1,10 @@
ARG HOME=/home
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@mcornaton mcornaton left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

group 'Build'
description 'Builds an OCI image from a Dockerfile.'
commandLine("sh", "-c", "docker build -f docker/Dockerfile-local --build-arg spring_boot_jar=$bootJarPath -t $ociImageName:$gitShortCommit ."
commandLine("sh", "-c", "docker build -f docker/Dockerfile --build-arg spring_boot_jar=$bootJarPath -t $ociImageName:$gitShortCommit ."
Copy link
Member

Choose a reason for hiding this comment

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

No palantir docker extension anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was removed to converge between all repositories and to be able to quickly copy/paste the docker build command if needed.

@jbern0rd jbern0rd merged commit 65ecc5b into develop Feb 11, 2022
@jbern0rd jbern0rd deleted the feature/create-library-project branch February 11, 2022 09:21
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.

4 participants