Generator-jhipster: Rename the default branch from `master` to `main`

Created on 15 Jun 2020  ·  62Comments  ·  Source: jhipster/generator-jhipster

Hey let's rename the master branch in JHipster repos and beat GitHub to it

@jhipster/developers

Context: https://www.zdnet.com/article/github-to-replace-master-with-alternative-term-to-avoid-slavery-references/#:~:text=GitHub%20is%20working%20on%20replacing,its%20CEO%20said%20on%20Friday.

$$ bug-bounty $$ $500 area

Most helpful comment

Thank you so much for your hard work @pascalgrimaud

All 62 comments

@deepu105 : I didn't follow closely the debate on social network, but is there a final official decision about this ?
For me, there is nothing urgent here. If "main" is not the new official name, we must rename to another one again

ok then let's wait to see what GitHub comes up with

Yes, let's use the new default, that's perfect!
I hope it won't be too much work on the currently opened pull requests.

Mark from the Spring Data Team has already build a script which renames the primary branch to main for all repositories under one org. https://twitter.com/mp911de/status/1271785736547729408 https://github.com/mp911de/rename-to-main

I think I have read somewhere when doing it via gh api existing PRs are updated automatically, but I can't find the reference anymore.

Could we test this on the hipster-labs org first? Or on one of our own personal repos? If the PRs are updated automatically, then there's no need to think a lot about this, let's just do it.

Dave Syer got covered the retargeting of open PRs: https://github.com/dsyer/main-branch-switch

You can test this on jhipster-kotlin 👍

Reading https://twitter.com/lukaseder/status/1273187350911291393 I believe we will have the same issue. I know I'm doing this quite often, and I'm probably not the only one.
So at the very least we should check if we have some "master" references in our own documentation.

At least CONTRIBUTING.md must be changed (although it is not a link to the master branch)

IMO its worth all the trouble if it helps to improve diversity and helps to
fight systematic racism

On Wed, 17 Jun 2020, 2:06 pm Frederik Hahne, notifications@github.com
wrote:

At least CONTRIBUTING.md must be changed (although it is not a link to
the master branch)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/generator-jhipster/issues/11935#issuecomment-645333534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIOKF6KJPXQDWCN25QKRUDRXCWTRANCNFSM4N6AEDWQ
.

Definitely. We should do it. And if we miss some parts first it's easy to change it afterwards.

Just for reference, I believe by end of this year GitHub will support this natively and probably it's easier to wait until then;

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

And currently they have done parts of it; https://github.blog/changelog/2020-07-22-github-actions-better-support-for-alternative-default-branch-names/

I'm ready to do this now. It's soooooo easy. PRs will suffer, but maybe that's the motivation for us to merge stuff?!

https://www.hanselman.com/blog/EasilyRenameYourGitDefaultBranchFromMasterToMain.aspx

I think we should wait for https://github.com/github/renaming/ to have a good developer experience

We should wait for patch release v6.10.4 before doing this too
Then, we can start with small repositories before doing the main repository

I'll do this in jhipster-online after v6.10.4 is released (and end of year) and maybe we can use that as a test run to see if everything works correctly. 😄

Refer: https://github.com/jhipster/jhipster-online/issues/246

Done for https://github.com/jhipster/react-jhipster

Done for https://github.com/jhipster/ng-jhipster

Done for https://github.com/jhipster/jhipster-registry

Done for https://github.com/jhipster/jhipster-control-center

Done for https://github.com/jhipster/generator-jhipster-module

Done for https://github.com/jhipster/generator-jhipster-blueprint

Done for https://github.com/jhipster/jhipster-registry-sample-config

  • create branch main from master
  • protect main
  • edit existing pull requests

Done for https://github.com/jhipster/consul-config-loader

Done for https://github.com/jhipster/jhipster-artwork

Done for https://github.com/jhipster/jhipster-fisher-plugin

  • create branch main from master
  • protect main
  • edit existing pull requests

Done for https://github.com/jhipster/jhipster-oh-my-zsh-plugin

  • create branch main from master
  • protect main
  • edit existing pull requests

Done for https://github.com/jhipster/jdl-samples

  • create branch main from master
  • protect main
  • edit existing pull requests

Done for https://github.com/jhipster/jhipster-base

Done for https://github.com/jhipster/jhipster-guides

I thought we were gonna wait for GitHub to support this

I had some time this afternoon, looking the Roland Garros final :tennis: :-)

Done for https://github.com/jhipster/eslint-config-jhipster

Does the rename for JDL-samples require code changes since the JDL command reads from it?

On Oct 11, 2020, at 09:22, Pascal Grimaud notifications@github.com wrote:


Done for https://github.com/jhipster/eslint-config-jhipster

create branch main from master
protect main
edit existing pull requests
some updates: jhipster/eslint-config-jhipster#7

You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

yes probably @mraible but I didn't delete master branch so it should still work for now
I didn't start the work yet for generator-jhipster, it will be the 'final boss'

Done for https://github.com/jhipster/jhipster.github.io

Done for https://github.com/jhipster/jhipster (cc @DanielFran)

done for https://github.com/jhipster/generator-jhipster

These repos needs to be changed too:

prettier-java : done

cc @clementdessoude

svelte : done

cc @vishal423

dotnet : done

cc @nicolas63 @danielpetisme

IDE: I did it yesterday evening :-)

cc @colameo

JHipster Online : done

cc @SudharakaP

Quarkus : done

cc @danielpetisme @avdev4j @joewhite101

Kotlin : done

cc @sendilkumarn

Node.js : done

cc @amanganiello90

Micronaut

cc @JasonTypesCodes @atomfrede

Go :done

cc @deepu105

JDL Studio : no need

cc @deepu105 (strange src branch)

JOOQ : done

cc @mshima

CN : done

cc @PierreBesson

Archived / Deprecated repos

These repos don't need to change, as there will be archived for v7:

Same for old repos or deprecated repos

Thank you so much for your hard work @pascalgrimaud

I'm taking those from Quarkus.

Yes, I done for :

But I haven't settings permissions for the last in order to change the default.

@deepu105 Could you help me?

@amanganiello90 : I added you as admin for the repo https://github.com/jhipster/generator-jhipster-nodejs

I'll update the Go repos. JDL studio doesn't need to change as it doesn't have a master branch and src contains source and gh-pages contain the build

I updated generator-jhipster-ionic to use main as the default branch.

https://github.com/oktadeveloper/generator-jhipster-ionic https://github.com/oktadeveloper/generator-jhipster-ionic

On Oct 12, 2020, at 3:05 AM, Deepu K Sasidharan notifications@github.com wrote:

I'll update the Go repos. JDL studio doesn't need to change as it doesn't have a master branch and src contains source and gh-pages contain the build


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jhipster/generator-jhipster/issues/11935#issuecomment-706987712, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAELZBFUJOWGTV4M45ZPZDSKLBGDANCNFSM4N6AEDWQ.

I forgot Azure Pipelines, so I just configure it to point to main branch instead of master, for these projects:

  • generator-jhipster
  • jhipster
    It means all recent PRs don't have Azure Pipelines -> cc @mshima

All other projects should use GitHub Actions instead of Azure Pipelines.
If someone needs to configure Azure Pipelines for one of these projects, don't hesitate to ping me.

It appears that neither @atomfrede @mraible or myself have rights to change the default branch on https://github.com/jhipster/generator-jhipster-micronaut/

See: https://github.com/jhipster/generator-jhipster-micronaut/issues/191

@JasonTypesCodes : just added you to admin team for Micronaut projects

@mshima : I created a new team jOOQ admins and added you

@mshima : I created a new team jOOQ admins and added you

Done.

@pascalgrimaud Everything has been migrated.
Can this be closed?

no, unfortunately, it's not finished yet...

Still to do on my side:

  • check all Sonar analysis
  • check all configuration at Azure DevOps (there are a lot of projects...), unless we decide to migrate to GitHub Actions

@SudharakaP @pascalgrimaud FYI the jhipser-bot used in the JHipster online is still creating master branch we creating a new project.

I don't think we noticed that ;)

@avdev4j : I think it's because this PR https://github.com/jhipster/jhipster-online/pull/255 is not deployed to GCP yet

great !

What was done:

  • Azure DevOps should be configured correctly for active project (JHipster Kotlin)
  • All SonarCloud projects should use main branch
  • at Docker Hub:

    • jhipster

    • jhipster-online

    • jhipster-registry

    • consul-config-loader

    • jhipster-control-center

I think all work is done so the ticket can be closed !

The last thing to do will be to delete the master branch
But I prefer to do it next year (in january 2021) so all our users can update their module / blueprint

Was this page helpful?
0 / 5 - 0 ratings

Related issues

hdurix picture hdurix  ·  58Comments

deepu105 picture deepu105  ·  56Comments

abhinav910 picture abhinav910  ·  50Comments

deepu105 picture deepu105  ·  75Comments

loydjayme25 picture loydjayme25  ·  71Comments