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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update references "master" to "main" #2172

Merged
merged 3 commits into from May 26, 2021
Merged

Conversation

axel-lauer
Copy link
Contributor

@axel-lauer axel-lauer commented May 17, 2021

This PR updates all references to "master" e.g. in the documentation and readme files (master branch) to "main" (the new GitHub standard, see https://github.com/github/renaming). See also corresponding PR in ESMValCore: ESMValGroup/ESMValCore#1132.

Closes ESMValGroup/ESMValCore#1031


Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 馃洜 Technical or 馃И Scientific review.

@valeriupredoi
Copy link
Contributor

cheers @axel-lauer 馃嵑 I've now fixed the automated tests to go from master to main and did a grep "master" -r . in the rootdir, it looks good to me 馃憤

Copy link
Contributor

@valeriupredoi valeriupredoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here, cheers @axel-lauer -> @bouweandela please have a looksee 馃憤 馃嵑

Copy link
Member

@bouweandela bouweandela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for taking care of this @axel-lauer!

@valeriupredoi valeriupredoi marked this pull request as ready for review May 26, 2021 15:19
@valeriupredoi
Copy link
Contributor

@axel-lauer I changed this PR to RfR and both Bouwe and meself have approved it, cheers! Gonna merge now, @bouweandela what say you, delete master for good after merge?

@valeriupredoi valeriupredoi merged commit 3ae1643 into master May 26, 2021
@valeriupredoi valeriupredoi deleted the update_master2main branch May 26, 2021 15:20
@bouweandela
Copy link
Member

The idea is to rename master to main, not delete it. See here for instructions and information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch main branch from "master" to "main"
4 participants