Skip to content

Split information about OWNERS out of GOVERNANCE.md#9205

Merged
mrjoro merged 6 commits intoampproject:masterfrom
mrjoro:mrjoro-owners-doc
May 8, 2017
Merged

Split information about OWNERS out of GOVERNANCE.md#9205
mrjoro merged 6 commits intoampproject:masterfrom
mrjoro:mrjoro-owners-doc

Conversation

@mrjoro
Copy link
Copy Markdown
Member

@mrjoro mrjoro commented May 8, 2017

Implements a fix for issue #8183.

This separates documentation on the approvals a PR requires + the concept of OWNERS out of GOVERNANCE which now focuses more on what the Core Committers are/etc.

cc @cramforce @erwinmombay

@mrjoro mrjoro added this to the Docs Updates milestone May 8, 2017
@mrjoro mrjoro requested a review from aghassemi May 8, 2017 16:51
GOVERNANCE.md Outdated
* In the unlikely event that there are no more Core Committers, Google Inc. will appoint a new Tech Lead.
* Significant feature development and changes to AMP require following the ["Intent to implement"](./CONTRIBUTING.md#feature-development) process including approval from the Tech Lead and one Core Committer.
* Significant feature development and changes to AMP require following the ["Intent to implement"](./CONTRIBUTING.md#contributing-features) process including approval from the Tech Lead and one Core Committer.
* Before contributions can be merged into the AMP Project approval must be given by an [Owner and a Core Committer](./contributing/owners-and-committers.md)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

needs comma? ...AMP Project, approval must be given...

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.

Done.

@mrjoro mrjoro merged commit a823e06 into ampproject:master May 8, 2017
@mrjoro mrjoro deleted the mrjoro-owners-doc branch May 8, 2017 23:41
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.

5 participants