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

Renaming master to main #68

Closed
jakirkham opened this issue Jun 23, 2020 · 25 comments
Closed

Renaming master to main #68

jakirkham opened this issue Jun 23, 2020 · 25 comments

Comments

@jakirkham
Copy link
Member

Would like to suggest we rename our branches from master to main. The term master can be problematic and is something even GitHub themselves is moving away from. Suspect beyond doing the work itself, the change has minimal (if any) long term maintenance associated. Would be interested in hearing others thoughts on this 🙂

@TomAugspurger
Copy link
Member

Some discussion on this for Python: https://discuss.python.org/t/communitys-take-on-changing-master-branch-to-main/4462

I'm in favor of changing this, but would happily let other projects move first and figure out what breaks :)

@jacobtomlinson
Copy link
Member

I support terminology changes like this. Doing a quick search for terms like whitelist and blacklist and highlight a few things we should think about changing.

We recently had to resolve an issue where Bokeh renamed whitelist to allowlist in a non-backward compatible way. I propose we follow suit but perhaps consider making it backward compatible for a while to avoid breaking stuff for other folks.

@jsignell
Copy link
Member

I agree that we should make this change.

I don't love the idea of making the whitelist/allowlist change backwards compatible unless we have a very short and clearly defined timeline for getting rid of it entirely. Breaking some things might be considered part of the goal for this work.

@jacobtomlinson
Copy link
Member

Breaking some things might be considered part of the goal for this work.

Interesting I hadn't considered that.

@TomAugspurger
Copy link
Member

TomAugspurger commented Jul 15, 2020 via email

@jacobtomlinson
Copy link
Member

Other discussions that I could find:

Of those it seems only Mozilla has gone ahead with the change already.

@jsignell
Copy link
Member

Definitely not the same as OSS but we made the change at Saturn and it was pretty seamless.

@TomAugspurger
Copy link
Member

FYI: https://github.com/github/renaming/.

If you haven’t moved yet, we recommend not moving right now, and waiting until later this year. We’re investing in tools to make the renaming the default branch of an existing repository a seamless experience for both maintainers and contributors.

@bryevdv
Copy link

bryevdv commented Aug 25, 2020

FWIW we did just start this, beginning with some peripheral repos, and so far things have been relatively painless.

@jacobtomlinson
Copy link
Member

It was raised in dask/dask#6922 that there are other words such as chain that we may want to consider changing for similar reasons. Diverting that discussion here to be continued.

@jsignell
Copy link
Member

I think Github has now implemented all the helper utils for migrating from master to main: https://github.com/github/renaming

I am down to help with any work that this change would entail.

@jrbourbeau
Copy link
Member

I'm also +1 for migrating to main and am around to help. Following up on #68 (comment), is there any objection to trying this out on a smaller project first (e.g. dask-examples) as a test before moving onto repos like dask and distributed with more traffic?

FWIW scikit-learn made the switch to main recently https://twitter.com/scikit_learn/status/1352677206972051456

@jacobtomlinson
Copy link
Member

Would probably be good to track this for subprojects in the same way we are doing for the GitHub Actions migration.

Also side note I wrote a blog post a while back that folks may find helpful on setting up aliases to check out and pull the default branch. I personally work on projects that use master, main, gh-pages, next, etc. So a smart alias is helpful.

@jsignell
Copy link
Member

jsignell commented Jan 27, 2021

@jacobtomlinson
Copy link
Member

jacobtomlinson commented Jan 27, 2021

Quick FYI GitHub helpfully lets you know the default branch has changed and you need to run some commands to update your local clone.

However it doesn't mention that you need to head to your fork and rename the default branch there first otherwise those commands will fail.

image

@jrbourbeau
Copy link
Member

Almost all of the repos in the Dask org have either made the migration from master -> main or have been archived. So far there haven't been any reported issues (that I'm aware of) as a result of the branch name change. Given this we're planning to move over dask/dask and dask/distributed soon. Just wanted to give folks tracking this issue a heads up

@jsignell
Copy link
Member

jsignell commented Mar 10, 2021

@jacobtomlinson just found that read the docs caches the name of the default branch, so project maintainers may need to verify that the correct branch is being built. I am opening issues to track this now:

@jsignell
Copy link
Member

I'm not sure if marketing uses read the docs.

@jrbourbeau
Copy link
Member

It does and I've pointed RTD to main

@jsignell
Copy link
Member

Ok then I think the list I have above is complete

@jakirkham
Copy link
Member Author

All branches listed have been updated.

Down to the RTD issues listed above. Only a couple people have permissions on those. Tried pinging for help.

@jakirkham
Copy link
Member Author

jakirkham commented Dec 6, 2022

Think only dask-glm remains. Suggesting archiving it ( dask/dask-glm#99 ) as it appears inactive

Edit: This is now archived

@phobson
Copy link

phobson commented Dec 7, 2022

@jakirkham we recently took ownership of crick from Jim. We should rename that too. I can lead that.

@jakirkham
Copy link
Member Author

jakirkham commented Dec 7, 2022

Renaming the branch is straightforward. Happy to do that

There seem to only be a few references to master in the repo. Though those seem unrelated to the project itself. Please let me know if I'm missing anything

Edit: Went ahead and renamed the branch

@jakirkham
Copy link
Member Author

Closing as completed

@jakirkham jakirkham unpinned this issue Dec 8, 2022
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

No branches or pull requests

7 participants