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

Master to Main README Update #132

Merged
merged 2 commits into from
Jul 20, 2021
Merged

Master to Main README Update #132

merged 2 commits into from
Jul 20, 2021

Conversation

mcleo-d
Copy link
Member

@mcleo-d mcleo-d commented Jul 20, 2021

Description

This PR updates the CSC README.md to update repo visitors of the recent branch change from master to main.

README Addition

Important Notice

The CSC default branch has been renamed.

master is now named main

If you have a local clone, you can update it by running:

git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

Closes #119

@mcleo-d mcleo-d self-assigned this Jul 20, 2021
@mcleo-d mcleo-d added the ready for review Item ready for review by the wider community label Jul 20, 2021
Copy link
Contributor

@TLATER TLATER left a comment

Choose a reason for hiding this comment

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

LGTM! People with more intricate local branches may want to use the script here: #119 (comment)

But I'll grant that that's too complex for a little readme entry :)

@TLATER
Copy link
Contributor

TLATER commented Jul 20, 2021

As a minor nit, if you update the commit message to include Closes #119 in the body, it would automatically close that issue once it lands (which I think we want), and help with context for anyone diving into the history in the future.

@mcleo-d
Copy link
Member Author

mcleo-d commented Jul 20, 2021

As a minor nit, if you update the commit message to include Closes #119 in the body, it would automatically close that issue once it lands (which I think we want), and help with context for anyone diving into the history in the future.

Thanks for clarifying the closes @TLATER. I wasn't sure if there were more tasks to complete on that issue first, but now I know we're good! 👍🏻

@mcleo-d
Copy link
Member Author

mcleo-d commented Jul 20, 2021

LGTM! People with more intricate local branches may want to use the script here: #119 (comment)

But I'll grant that that's too complex for a little readme entry :)

@TLATER - I have added a link to your comment on commit 8d0ac2d

@mcleo-d mcleo-d requested a review from TLATER July 20, 2021 11:10
@peterrhysthomas peterrhysthomas merged commit c26a293 into main Jul 20, 2021
@abdullahgarcia abdullahgarcia deleted the master-to-main-update branch October 7, 2021 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-present ready for review Item ready for review by the wider community
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Rename CSC primary branch to follow best practices
3 participants