-
-
Notifications
You must be signed in to change notification settings - Fork 32
Remove references to OTC and OMC #116
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
Conversation
Also remove inapplicable policies.
|
@mattcaswell fixup pushed accepting all your proposed changes. |
t-j-h
left a comment
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.
A summary of the changes in terms of whether they require agreement ("and") or either organisation can act unilaterally "or".
I don't think we have the mix correct or consistent here. I think the conservative approach is to have "and" and the liberal approach is to have "or" and we really should be consistent.
I would expect "or" or "and" for everything against each policy - and the branch policy is a mix. I would be happy with all being "and" or all being "or" and that "or" actually reflects more of what we have been doing - where either the corporation or the foundation can make the decision - we seek consensus, but consensus is not required if either organisation feels that a specific issue better represents the needs of their community focus.
I think the stronger controls (the "and" usage) on the release requirements does make sense.
Branch Policy
Override for backfits: or
API/ABI breakage: or
Future major branch updates: and
Future minor branch features: or
Future minor branch updates: or
Branch creation: and
Release Requirements
Release objectives set: and
First beta: engineering-managers and
Major and minor releases: engineering-managers and
Responsibility for meeting requirements: engineering-managers and
Performance testing requirements: engineering-managers and
|
@t-j-h @mattcaswell please look again |
t-j-h
left a comment
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.
Looks good. Thanks for the updates!
policies/branch-policy.md
Outdated
| - According to [stable release update policy] only bug fixes and | ||
| documentation changes are allowed. | ||
| - By exception given by OMC also other types of pull requests can be merged. | ||
| - By exception given by the OpenSSL Foundation and the OpenSSL Corporation |
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.
Not sure if in all cases, should be "and" between OSF and OSS, as in some sentences makes sense also "or"
|
This was merged manually. |
Also remove inapplicable policies.