Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

Use HEAD instead of master as current #8

Merged
merged 1 commit into from
Feb 20, 2019

Conversation

leny
Copy link
Contributor

@leny leny commented Feb 20, 2019

Hi,

Got into a small issue when using this action within a repo with no master branch (or with a default branch not called master), due to the following line:

git push --force $REMOTE_REPO master:$REMOTE_BRANCH

By using HEAD instead of directly mention master, we can fix this.

@maxheld83
Copy link
Owner

yeah, thanks @leny that makes a ton more sense.
If people want to only deploy from master (or whatever), they should use a filter action before.
This action should just deploy from whatever branch happens to be in github/workspace at the time.

@maxheld83 maxheld83 merged commit eced9e9 into maxheld83:master Feb 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants