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

support "main" as default branch #9

Merged
merged 1 commit into from
Oct 23, 2020

Conversation

dbushong
Copy link
Member

This change is non-breaking as it still falls back on "master".

A number of projects are switching to using "main" as their default branch; this change supports "main" if that branch exists, and falls back on "master" if it doesn't. (But you have to have one of them)

Keeping in the spirit of this tool's zero-configuration and opinionated branch naming, it does not support any arbitrary main branch, just those two.


This PR was started by: git wf pr

non-breaking as it still falls back on "master"
@@ -1,16 +1,22 @@
# Branch Workflow CLI

A cli that provides a set of `git wf` subcommands which simplify dealing with
feature branches & GitHub pull requests. Does not require a GH API token, as
feature branches & GitHub pull requests. Does not require a GH API token, as
Copy link
Member Author

Choose a reason for hiding this comment

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

Ran prettier on this (somewhat unintentionally), hence all the formatting changes.


log('merge-back is clean');

if (origBranch !== 'master') await git.checkout(origBranch);

return true;
Copy link
Member Author

Choose a reason for hiding this comment

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

not sure what this was for originally, but tsc didn't like it

@dbushong dbushong merged commit 6a5375c into master Oct 23, 2020
@dbushong dbushong deleted the dbushong/feature/master/support-main branch October 23, 2020 17:16
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.

3 participants