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

Handle case when using detached branch (as Travis CI does) #194

Merged
merged 1 commit into from Jan 9, 2013

Conversation

mojavelinux
Copy link
Member

If the current branch is detached, handle it gracefully. An attempt to checkout the branch '(no branch)' will fail. In this case, the current commit sha (retrieved from revparse HEAD) must be used as the return branch.

This problem is observed when Travis CI uses this deployer.

@buildhive
Copy link

awestruct » awestruct #99 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@lance
Copy link
Member

lance commented Jan 9, 2013

Just curious if you're doing deployment from CI.

lance added a commit that referenced this pull request Jan 9, 2013
Handle case when using detached branch (as Travis CI does)
@lance lance merged commit 4927d36 into awestruct:master Jan 9, 2013
@mojavelinux
Copy link
Member Author

Yes, on one of my sites I'm doing deployment from Travis CI. I plan to
document the process some point soon.

Sent from my CyanogenMod-powered
Android device, an open platform for
carriers, developers and consumers.
On Jan 8, 2013 5:16 PM, "Lance Ball" notifications@github.com wrote:

Just curious if you're doing deployment from CI.


Reply to this email directly or view it on GitHubhttps://github.com//pull/194#issuecomment-12024855.

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