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

Do not try to setup missing deployment settings for Docker. #251

Merged
merged 1 commit into from
May 16, 2014
Merged

Do not try to setup missing deployment settings for Docker. #251

merged 1 commit into from
May 16, 2014

Conversation

fiadliel
Copy link
Contributor

Docker has no tasks for packageBin/etc. It should not be mentioned
in the deploymentSettings definition. Otherwise, adding
deploymentSettings to your build results in:
Reference to undefined setting:

docker:packageBin from docker:$local (build.sbt:3)

Docker has no tasks for packageBin/etc. It should not be mentioned
in the deploymentSettings definition. Otherwise, adding
deploymentSettings to your build results in:
Reference to undefined setting:

  docker:packageBin from docker:$local (build.sbt:3)
@jsuereth
Copy link
Member

LGTM, once travis has a chance to verify.

@timperrett
Copy link

This is also affecting me; Travis CI says its good to go :-)

muuki88 added a commit that referenced this pull request May 16, 2014
Do not try to setup missing deployment settings for Docker.
@muuki88 muuki88 merged commit 75b9dd0 into sbt:master May 16, 2014
@muuki88
Copy link
Contributor

muuki88 commented May 16, 2014

Thanks a lot for your effort @garycoady 👍

@muuki88 muuki88 added this to the 0.8.0 - windows services milestone May 16, 2014
@fiadliel fiadliel deleted the fix/no-docker-publish branch May 16, 2014 09:20
@muuki88 muuki88 added the docker label Jun 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants