Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

plugin using 'main' default branch ends up in error during plugn update #71

Closed
Cactusbone opened this issue Mar 2, 2022 · 2 comments · Fixed by #141
Closed

plugin using 'main' default branch ends up in error during plugn update #71

Cactusbone opened this issue Mar 2, 2022 · 2 comments · Fixed by #141

Comments

@Cactusbone
Copy link

Cactusbone commented Mar 2, 2022

See

git checkout master &> /dev/null

git checkout master &> /dev/null does not work when no 'master' branch exists. (newly created github projects uses main as default)

Seen here: dokku/dokku#5041

However to work around it, I changed my default branch to 'master'.

Using something like git checkout refs/remotes/origin/HEAD should work

@josegonzalez
Copy link
Member

Good catch! Mind making an MR for this?

@mattmess1221
Copy link
Contributor

mattmess1221 commented Mar 26, 2024

I think I can get this fixed.

Is there a reason a lot of the the git commands end in &>/dev/null instead of >/dev/null? Unless it's some obscure syntax, I don't see much of a reason to run all the commands in the background. It might even be detrimental when you checkout a new commit before the pull finishes. Oh, I see. It redirects stdout and stderr.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants