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

Standardize on main as default branch name #8

Open
7 of 47 tasks
genehack opened this issue May 24, 2024 · 9 comments
Open
7 of 47 tasks

Standardize on main as default branch name #8

genehack opened this issue May 24, 2024 · 9 comments
Assignees
Labels
enhancement New feature or request

Comments

@genehack
Copy link
Contributor

genehack commented May 24, 2024

todo

has issue created

master renamed to main

@ivan-aksamentov
Copy link
Member

Is this a proposal or is decision has already been made?

@genehack
Copy link
Contributor Author

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.

@ivan-aksamentov
Copy link
Member

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.

@genehack
Copy link
Contributor Author

I see. I guess I am late to the party then. But just want to say that I am strongly against.

Could you explain why?

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.

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.

@ivan-aksamentov
Copy link
Member

Could you explain why?

Many things, but on top of my head:

  • if it's working don't fix it. It's been 3-4 years and we are doing okay
  • a lot of work without advancing science or technology further
  • 'standartization' might be defined too vague here: master branch in Nextclade does not have the same meaning as in ncov and both are different from nextstrain.org. Different project requirements might require different branching strategies.

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?

@genehack
Copy link
Contributor Author

* if it's working don't fix it. It's been 3-4 years and we are doing okay

Okay.

* a lot of work without advancing science or technology further

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.

* 'standartization' might be defined too vague here: master branch in Nextclade does not have the same meaning as in ncov and both are different from nextstrain.org. Different project requirements might require different branching strategies.

This is not about branching strategies; it is about the name of the default branch.

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?

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 main while others have master causes extra coginive load. When I raised the issue of why we weren't consistent, and why we hadn't aligned with what git itself defaults to these days, the outcome of the ensuing conversation was this issue.

@victorlin
Copy link
Member

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.

image

@victorlin
Copy link
Member

Much of the work involved here will be removing/replacing hardcoded references to master. There are a few options here (HEAD, @, -) discussed in various places (nextstrain/pathogen-repo-guide#41 (comment) nextstrain/auspice#1660 (comment)) which might be worth reiterating here. When choosing, these are the differences:

  • - redirects to the default branch and seems to only work for certain endpoints.
  • HEAD and @ preserve the URL and show the latest commit on the default branch in the GitHub branch/tag selector. These work for more endpoints than -.

@genehack
Copy link
Contributor Author

Much of the work involved here will be removing/replacing hardcoded references to master. There are a few options here (HEAD, @, -) discussed in various places (nextstrain/pathogen-repo-guide#41 (comment) nextstrain/auspice#1660 (comment)) which might be worth reiterating here. When choosing, these are the differences:

* `-` redirects to the default branch and seems to only work for certain endpoints.

* `HEAD` and `@` preserve the URL and show the latest commit on the default branch in the GitHub branch/tag selector. These work for more endpoints than `-`.

I used @ for the above reasons, and picked it over HEAD to avoid any confusion with other places HEAD might get used in a Git context.

@genehack genehack transferred this issue from another repository Nov 22, 2024
@genehack genehack transferred this issue from another repository Nov 22, 2024
@genehack genehack mentioned this issue Nov 22, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants