-
Notifications
You must be signed in to change notification settings - Fork 0
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
Standardize on main
as default branch name
#8
Comments
Is this a proposal or is decision has already been made? |
It was discussed — I'm sorry, I'm not finding the conversation, but there was a consensus that it was time to start rolling this out. |
I see. I guess I am late to the party then. But just want to say that I am strongly against. Regarding specifically Nextclade and Nextclade Data, they rely on GitOps heavily (docs) and this rename might require changes to the underlying infra (including domain names and SSL certs). Nothing impossible, but is likely more time-consuming than for most repos. |
Could you explain why?
Thanks for surfacing this; this type of thing is why I've been creating individual issues in each repo, so that such additional work can be identified and tracked. |
Many things, but on top of my head:
I am not touching political side of things, but I can think of a couple of reasons too. Was it significant in decision-making here? |
Okay.
In many cases it is an absolutely trivial change. Again, the reason for issues in every repo is so, in the cases where it is not trivial, or even in cases where the decision is ultimately made to not change, there will be documentation of what needs to change, or why the change was deemed not useful.
This is not about branching strategies; it is about the name of the default branch.
I'm not sure what you about "political side". As a new person joining the organization, having some of the repos have a default branch of |
I noticed that after nextstrain/zika-tutorial-nextflow#6, the repo's main page showed a tooltip about default branch name change. Not that this reduces any work that would need to be done, but it was nice to see. |
Much of the work involved here will be removing/replacing hardcoded references to
|
I used |
todo
has issue created
master
renamed tomain
The text was updated successfully, but these errors were encountered: