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

Conditional support for writing a REVISION file #260

Closed

Conversation

pjaspers
Copy link

As mentioned in #72 , having the exact sha of the deployed version of
an application available in the root of the project can be very
handy. For example: it's easy to retrieve the deployed version when the
file is there.

If the user now sets the git_revision_file in her deploy.rb config
file:

set :git_revision_file, 'REVISION'

A file named 'REVISION' will be written during deployment which contains
the sha of the project being deployed (obtained via git rev-parse HEAD).

Existing projects won't noticed any difference since the
git_revision_file flag must be set to have any effect on the deploy.

As mentioned in mina-deploy#72 [0], having the exact sha of the deployed version of
an application available in the root of the project can be very
handy. For example: it's easy to retrieve the deployed version when the
file is there.

If the user now sets the `git_revision_file` in her `deploy.rb` config
file:

```
set :git_revision_file, 'REVISION'
```

A file named 'REVISION' will be written during deployment which contains
the sha of the project being deployed (obtained via `git rev-parse
HEAD`).

Existing projects won't noticed any difference since the
`git_revision_file` flag must be set to have any effect on the deploy.
Since it would set nil and then + it, it would throw an error, now
defaulting to "".
@davidbrewer
Copy link

Ooh, yes please. I was just about to hack this in myself.

@d4be4st
Copy link
Member

d4be4st commented Jul 5, 2015

duplicate of #306. closing

@d4be4st d4be4st closed this Jul 5, 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.

None yet

3 participants