-
Notifications
You must be signed in to change notification settings - Fork 43
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
Update README to reflect making main the default branch #848
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
NOTE: this pull request branch is against the |
Probably need to add something to the README about our plan for keeping the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes and the new blog post process looks great to me, thanks Gabriel!
I won't merge this now, since it'd be good for at least one of @trallard or @pavithraes to also have a look.
Great! I just changed the default branch in the repo settings to |
Thanks! I went through the updates as well, and all looks good. :) |
This PR updates the README to reflect the discussion on a recent release PR (copy-pasted below).
I also reworded some things and moved two sections to the top:
Once it is approved, right after merging it, I will change the setting in GitHub to make
main
the default branch.Link: updated README
As for getting rid of the two-stage deploy process, here are some thoughts:
main
to be the default branch so that blog authors branch off main when working on a blog postdevelop
branch in place for riskier changes to the repo that should go through the two-stage deploy processdevelop
branch withmain
. Seems to me thatmain
should be the source of truth anddevelop
should mirrormain
. (The alternative would be merge commits from main to the develop branch but that sounds messy in the long run.)