这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@lvillani
Copy link
Contributor

The rationale here is to make it possible to git push dokku master a base image that is used by other applications that are effectively being deployed.

Used in this manner, Dokku becomes a lightweight alternative to a private Docker registry.

@michaelshobbs
Copy link
Member

You'll need to rebase this in order to get tests to pass.

Copy link
Member

Choose a reason for hiding this comment

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

We'll probably want to support global with an override. See this as an example: https://github.com/dokku/dokku/blob/v0.4.6/dokku#L96-L102

@lvillani
Copy link
Contributor Author

Rebased and updated to add support for global setting. For some reason the test are still failing though.

@lvillani
Copy link
Contributor Author

Ah, it seems it was a temporary problem. Looks green now.

Copy link
Member

Choose a reason for hiding this comment

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

Sorry I missed this in the first pass, but following our coding style guide, can you swap out the deprecated [ for the preferred [[? Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right! I'll try to fix it ASAP (I'm on a train and have bad connectivity, atm)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be OK now.

Copy link
Member

Choose a reason for hiding this comment

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

Excellent! Thank you

michaelshobbs added a commit that referenced this pull request Dec 22, 2015
Make it possible to skip a deploy
@michaelshobbs michaelshobbs merged commit 32f1bc5 into dokku:master Dec 22, 2015
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