Skip to content

Use different branch name as the default#198

Closed
techknowlogick wants to merge 1 commit intocharmbracelet:mainfrom
techknowlogick:patch-1
Closed

Use different branch name as the default#198
techknowlogick wants to merge 1 commit intocharmbracelet:mainfrom
techknowlogick:patch-1

Conversation

@techknowlogick
Copy link

To align with the default from various other forges (GitHub, GitLab, Gitea, etc..), and the git client itself. I've updated the default branch name for new repos to match.

To align with the default from various other forges (GitHub, GitLab, Gitea, etc..), and the git client itself. I've updated the default branch name for new repos to match.
@bashbunni bashbunni self-assigned this Mar 27, 2023
@bashbunni
Copy link
Contributor

Hey @techknowlogick this is definitely something people have expressed interest in. Unfortunately, this is still initializing the config repo with a master branch instead of main. Were you able to get this working on your machine?

@techknowlogick
Copy link
Author

Thanks for testing out this PR @bashbunni, I dove deeper into the upstream library and found that it was due to go-git using a default ref, rather than one that soft-serve sets.
I submitted a PR upstream go-git/go-git#727 to be use the updated ref

@bashbunni
Copy link
Contributor

Ah amazing @techknowlogick thanks for digging into that further and finding the root cause 🙏

@decentral1se
Copy link

This is solved in 0.5.0 afaiu 👍

@techknowlogick
Copy link
Author

Thanks @decentral1se. I'll close this PR :)

@techknowlogick techknowlogick deleted the patch-1 branch May 3, 2023 01:29
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.

3 participants