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

[Travis] Allow detached head when parsing master ref. #2286

Merged
merged 1 commit into from
Mar 28, 2015

Conversation

adunkman
Copy link
Contributor

@harvesthq/chosen-developers

If we have checked out a commit instead of a branch name, git is in what’s known as “detached head” state.

This is what Travis does all the time — see the latest build output.

This means that we were never considered to be on the branch “master”, and therefore were never publishing bower-chosen.

We can tweak the git command we’re calling to support either checked out branch names or detached heads — as long as we’re at the commit for a commit that is the head of master, we’re peachykeen!

If we have checked out a commit instead of a branch name, git is in what’s known as “detached head” state.

This is what Travis does all the time.

This means that we were never considered to be on the branch “master”, and therefore were never publishing bower-chosen.

We can tweak the git command we’re calling to support either checked out branch names or detached heads — as long as we’re at the commit for a commit that is the head of master, we’re peachykeen!
@pfiller
Copy link
Contributor

pfiller commented Mar 28, 2015

👍 boom. Thanks, @adunkman!!!!

pfiller added a commit that referenced this pull request Mar 28, 2015
[Travis] Allow detached head when parsing master ref.
@pfiller pfiller merged commit 7eafb45 into master Mar 28, 2015
@pfiller pfiller deleted the allow-detached-head branch March 28, 2015 03:33
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