-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
[#517] Create projects with main branch #518
Conversation
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.
There is no place for me here... I will choose the truth I like.
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.
Great work! I left one comment with one more patch we need to do, I guess. But overall looks great 👍🏻 Gigantic work! 🦶🏻
README.md
Outdated
@@ -789,8 +789,8 @@ If you'd like to take part in the development processes, here are a few things t | |||
|
|||
* Summoner is a multi-package project which has two packages inside: `summoner-cli` and `summoner-tui`. We are supporting the same functionality in both of them, so, if it's possible, you should add/change the code in both of the packages. | |||
* Summoner uses the custom prelude [`relude`](https://github.com/kowainik/relude). | |||
* We are using the [Kowainik style guide](https://github.com/kowainik/org/blob/master/style-guide.md#haskell-style-guide). | |||
* To sum up, [here is the Contributing guide](https://github.com/kowainik/.github/blob/master/CONTRIBUTING.md#contributing-to-the-kowainik-repositories) we use across the repositories. | |||
* We are using the [Kowainik style guide](https://github.com/kowainik/org/blob/main/style-guide.md#haskell-style-guide). |
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.
I just noticed, that this could be a link to our blog post, not the markdown file 🙂
@@ -296,7 +296,9 @@ doGithubCommands Settings{..} = do | |||
Just _ -> do | |||
isHubSuccess <- runHub repo | |||
if isHubSuccess | |||
then "git" ["push", "-u", "origin", "master"] | |||
then do | |||
"git" ["push", "-u", "origin", "main"] |
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.
I just googled and I see that the git init
command now takes the argument --initial-branch
to specify the default branch since git 2.28
. I guess, we can add this option to our git init
command above and update the README with the minimal git
version required for the tool to work.
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.
Superb! 👍🏻
Resolves #517