-
Notifications
You must be signed in to change notification settings - Fork 132
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
Consider changing the default branch to reflect the current git flow #1031
Comments
I agree with this. Setting the |
Make sense, I don't remember why is not dev the base branch. I also would like to know why you don't like this workflow, and possible alternatives. |
I fully support this! I saw there were some discussions on Discord recently. Was that already sorted out? also relevant on this context: #1030 (comment) |
@Fi3 did you get the chance to change |
This is mostly naming issues and I'm not really interested everything will work. What I'm interested in is:
Can you elaborate that point please? Ty |
Currently we merge to
dev
branch and in practice use it as the default branch, whilemain
is reserved as a release branch. This setup makes it necessary to update each PR opened againstdev
on each change indev
even if not conflicting with the opened PR. I suggest we setdev
as default branch in github settings to avoid that.As a side note, I am not a big fan of
dev
+main
workflow but thats a different discussion. While we have this setup already, lets make it as good as possible.@Fi3 would appreciate your input as it seems you have admin access to the repo
The text was updated successfully, but these errors were encountered: