Skip to content

Merge & Access Policy#790

Merged
michaelcullum merged 2 commits into
php-fig:masterfrom
michaelcullum:merge-policy
Aug 16, 2016
Merged

Merge & Access Policy#790
michaelcullum merged 2 commits into
php-fig:masterfrom
michaelcullum:merge-policy

Conversation

@michaelcullum
Copy link
Copy Markdown
Member

@michaelcullum michaelcullum commented Aug 3, 2016

Merge policy. We can also implement https://pullapprove.com/ as a kind of soft-enforcement (doesn't affect pushing after all, just the green button).

@php-fig/secretaries

Comment thread CONTRIBUTING.md Outdated

# Merge & Access Policy

All Editors, Coordinators and Sponsors of specifications in draft stage have
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not only in draft stage, I guess.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We remove access to the main repo after the draft/review stage but leave the groups intact for @mentions

@michaelcullum
Copy link
Copy Markdown
Member Author

Working to setup https://pullapprove.com

@michaelcullum
Copy link
Copy Markdown
Member Author

michaelcullum commented Aug 15, 2016

@php-fig @php-fig/psr-5 @php-fig/psr-8 @php-fig/psr-9-10 @php-fig/psr-11 @php-fig/psr-12 @php-fig/psr-13 @php-fig/psr-14 @php-fig/psr-15 @php-fig/psr-16 @php-fig/psr-17

Apologies for the mass mention, it seemed like an easier way to get your attention as I think most of you missed the mailing list email. Could you all please review this policy (and +1 the first post for acknowledgement).

There is a sort of forced implementation of who can merge what on this repository by a tool called pull approve. Essentially, it requires one of the WG to approve modifications to a PSRs file by them just saying 👍 in the PR or hoping on the pull approve UI. Secretaries can override all of these approvals. You can see the config for who it will let approve (and therefore merge) what here.

@michaelcullum michaelcullum merged commit b7b425b into php-fig:master Aug 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants