Skip to content

Rename main branch - #9

Merged
targos merged 1 commit into
masterfrom
move-to-main
Jul 2, 2021
Merged

targos merged 1 commit into
masterfrom
move-to-main

Conversation

@targos

@targos targos commented Jul 1, 2021

Copy link
Copy Markdown
Member

I'll do the actual renaming right before this is merged.

@canterberry

Copy link
Copy Markdown
Contributor

#7 updates the URLs in the README. I'll rebase that and my other PRs on main and resolve conflicts post-merge of this PR.

There may be tools referencing the master branch directly. If GitHub does not automatically redirect pulls from master such that they pull from main automatically (akin to HTTP 301), then on the master branch only, there may be a need for an informational post-pull git hook and README notice to inform users of the rename/deprecation.

Comment thread README.md Resolved

@canterberry canterberry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. One minor note on the repo URLs, but given there is a PR already in flight to address that, non-blocking for this PR.

Comment thread README.md Resolved
This allows us to rename the branch to "main".
@targos

targos commented Jul 2, 2021

Copy link
Copy Markdown
Member Author

Changed the links to use HEAD.

@targos
targos merged commit f2503f4 into master Jul 2, 2021
@targos
targos deleted the move-to-main branch July 2, 2021 11:45
@canterberry

Copy link
Copy Markdown
Contributor

@targos I pushed a main branch from d7dd522 just now, which matches the commit ref for master as well. I do not have privileges to set the default branch on this repo, so FYI should be ready to swap for anyone with the capability to edit that.

@targos

targos commented Jul 3, 2021

Copy link
Copy Markdown
Member Author

I just renamed the branch 👍🏻

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

Successfully merging this pull request may close these issues.

5 participants