-
Notifications
You must be signed in to change notification settings - Fork 1.2k
📖 Versioning and Branching Strategy #216
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
📖 Versioning and Branching Strategy #216
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: DirectXMan12 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@droot @pwittrock PTAL and let me know what you think |
|
||
### Breaking Changes | ||
|
||
Try to avoid breaking changes. They make life difficult for users, who |
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.
Can we provide a sed
operation to automatically fix any breaking changes with defaults that maintain the old behavior or provide some new default?
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.
Probably want more intelligent than sed, but it should be possible to do so
This adds in documentation on versioning, branching, and maintenance. There's still a couple of places where we have to create scripts and fill in process, but this lays the initial groundwork.
657d0aa
to
64a9833
Compare
New changes are detected. LGTM label has been removed. |
|
||
### Mantainers | ||
|
||
Don't be lazy, read the rest of this doc :-) |
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.
:)
### Contributors | ||
|
||
- All code PR must be labeled with :bug: (patch fixes), :sparkles: | ||
(backwards-compatible features), or :warning: (breaking changes) |
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.
Github PR template can help us here.
…ng-strategy 📖 Versioning and Branching Strategy
This is mostly a proposal at this point. I've started to outline the branch strategy we've loosely discussed in the past, with some motivations and discussion of alternatives.