-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Contributing branches and reverting #9577
Contributing branches and reverting #9577
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.
I first learned of this revert style from LLVM. It surprised me at first, but then I came to believe it is good.
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.
Thank you, this is important.
- _other_ | ||
|
||
Branches that do not conform to the above patterns should be feature branches. |
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.
IMHO those should not exist at all, and feature branches should come from forks. Since this documentation is descriptive rather than prescriptive, I won't block that with my desire to discuss this, but I'd still like to discuss it eventually.
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.
Forks in organizations are problematic because they can't be configured to allow contributions from upstream maintainers. We might suggest using personal forks.
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 also tend to use feature branches from the NixOS repo because it's the only sane way for having stacked PRs in GitHub (although that's a bit of an edge-case)
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.
Fwiw this makes no claim about the lifecycle of the feature branch. Any branch that isn't a maintenance branch or - I guess - a proof of concept should be short-lived.
I don't think we should focus any attention on these branches for the time being, while we have bigger fish to fry, such as the again increasing PR queue, and stabilization.
Co-authored-by: Valentin Gagarin <[email protected]>
The GitHub search is not fantastic but gets the job done.
Motivation
I consider the latter crucial for agility.
Let's review this in a meeting.
Context
Priorities
Add 👍 to pull requests you find important.