igraph / rigraph

igraph R package
https://r.igraph.org
551 stars 200 forks source link

Rename master to main? #611

Closed maelle closed 1 year ago

maelle commented 1 year ago

This fits with the DEI topic. :smile_cat:

maelle commented 1 year ago

@krlmlr

szhorvat commented 1 year ago

I strongly believe that in order for igraph to be a project where everyone can work together, the discussion here must have one clear focus: developing a network analysis library. This is what keeps things going smoothly.

There is a right place for everything, and this is not the place for making political statements, which this effectively is. I want this to be a place where people can work together even if their countries are at war with each other. This is achievable if we keep the focus on what is relevant for the project, and stay away from anything that elicits unproductive debate, especially alignment with specific political movements of specific countries.

ntamas commented 1 year ago

I don't think it's a political statement. It's a recognition that in certain countries / communities, words like master and slave might be controversial, while main is not. I don't give a sh*t about politics.

Note that Git and Github has already switched to using main instead of master as the default branch name. Doing the rename on our side has nearly zero cost to us (apart from maybe ~2 mins of inconvenience to rename our local checkout branches), so I don't have any objetions.

vtraag commented 1 year ago

Fully agree with @ntamas here.

There's been a switch to using main as the default in the community already a while ago, and I think it's perfectly reasonably for igraph to follow suit. We have similarly made changes in the code, preferring more neutral terminology, using for instance parent instead of father (although this is apparently not done everywhere yet).

I propose to switch from using the master branch to main in all igraph repositories, in order to avoid confusion.

iosonofabio commented 1 year ago

Let's just do it, it's a big deal for part of the community and basically zero chat for the rest 😉

I love zero cost changes

Cheers, Fabio

On Sat, Dec 17, 2022, at 08:24, Vincent Traag wrote:

Fully agree with @ntamas https://github.com/ntamas here.

There's been a switch to using main as the default in the community already a while ago, and I think it's perfectly reasonably for igraph to follow suit. We have similarly made changes in the code, preferring more neutral terminology, using for instance parent instead of father (although this is apparently not done everywhere yet).

I propose to switch from using the master branch to main in all igraph repositories, in order to avoid confusion.

— Reply to this email directly, view it on GitHub https://github.com/igraph/rigraph/issues/611#issuecomment-1355628060, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJFEAEW2NGPO2EUM76TNU3WNTMYHANCNFSM6AAAAAATAXXSAY. You are receiving this because you are subscribed to this thread.Message ID: @.***>

krlmlr commented 1 year ago

Thanks for considering.

There's an organization-wide setting on GitHub that allows defining the name of the main branch for new repositories, too. The corresponding local Git config is:

git config --global init.defaultBranch main

This is useful if you create a repository locally first and then push it to GitHub.

ntamas commented 1 year ago

In the R repository, I think we should simply wait until we are ready to get rid of the two-branch setup; we can then rename dev into main and delete master.

Right now probably there's still something missing from dev as remotes::install_github("igraph/rigraph@dev") prints an HTTP 404 error.

krlmlr commented 1 year ago

Just to confirm: are there any unresolved action items here?

ntamas commented 1 year ago

I don't think so.