{"action":"created","comment":{"url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/comments/169795700","pull_request_review_id":98381770,"id":169795700,"diff_hunk":"@@ -45,163 +61,211 @@ OEP Types\n * A **Process** proposal describes a change to how the Open edX community\n   functions.\n \n-* A **Best Practices** proposal describes a technology or implementation\n+* A **Best Practice** proposal describes a technology or implementation\n   choice that the Open edX community believes all applicable Open edX services\n   and/or libraries should use or follow.\n \n-* An **Architecture** proposal describes relationships between Open edX\n-  services and libraries, including their APIs and data formats.\n-\n-* A **Product Enhancement** proposal describes a significant enhancement to the\n-  functionality of the Open edX platform.\n+* An **Architecture** proposal describes a structure or framework for Open\n+  edX services or relationships between them.\n \n-Workflow\n-========\n+OEP Roles\n+=========\n \n-.. contents::\n-  :local:\n-  :depth: 1\n+Authors\n+-------\n \n-OEP Roles\n----------\n+Each OEP must have at least one Author: someone who writes the OEP using the\n+style and format described here, shepherds the discussions in the appropriate\n+forums, and attempts to build community consensus around the idea.  \n \n-Each OEP must have a Author: someone who writes the OEP using the style and\n-format described here, shepherds the discussions in the appropriate forums,\n-and attempts to build community consensus around the idea.  \n+Arbiter\n+-------\n \n-Each OEP also has an Arbiter (as described in `Initial Submission`_). The\n-Arbiter will be chosen by the edX Chief Architect (currently Eddie Fagin).  \n-The Author of an OEP will never be selected as the Arbiter of that OEP.\n+Each OEP also has an Arbiter (as described in `Step 4. Request an Arbiter`_). \n+The Arbiter will be chosen by the `edX Architecture Team`_. An Author of an OEP\n+will never be selected as the Arbiter of that OEP.\n \n The Arbiter will be the person making the final decision on whether the OEP \n should be Accepted, and as such, the Arbiter should be knowledgeable about \n the contents of the proposal, and willing to listen to arguments both for \n and against it by the rest of the community.\n \n-The Arbiter is also responsible for helping the Author move the proposal\n+The Arbiter is also responsible for helping the Authors move the proposal\n through the OEP process, providing technical and process expertise as needed.\n-The Arbiter also assists the Author to solicit feedback from the community on\n+The Arbiter also assists the Authors to solicit feedback from the community on\n the OEP, and for helping to move the OEP towards a final decision (whether that","path":"oeps/oep-0001.rst","position":137,"original_position":137,"commit_id":"0bf355d08a8d85e6b7adc61aa2f675cdfee65057","original_commit_id":"0bf355d08a8d85e6b7adc61aa2f675cdfee65057","user":{"login":"nasthagiri","id":6772826,"avatar_url":"https://avatars1.githubusercontent.com/u/6772826?v=4","gravatar_id":"","url":"https://api.github.com/users/nasthagiri","html_url":"https://github.com/nasthagiri","followers_url":"https://api.github.com/users/nasthagiri/followers","following_url":"https://api.github.com/users/nasthagiri/following{/other_user}","gists_url":"https://api.github.com/users/nasthagiri/gists{/gist_id}","starred_url":"https://api.github.com/users/nasthagiri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nasthagiri/subscriptions","organizations_url":"https://api.github.com/users/nasthagiri/orgs","repos_url":"https://api.github.com/users/nasthagiri/repos","events_url":"https://api.github.com/users/nasthagiri/events{/privacy}","received_events_url":"https://api.github.com/users/nasthagiri/received_events","type":"User","site_admin":false},"body":"fixed.","created_at":"2018-02-21T22:15:39Z","updated_at":"2018-02-21T22:15:39Z","html_url":"https://github.com/edx/open-edx-proposals/pull/53#discussion_r169795700","pull_request_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/comments/169795700"},"html":{"href":"https://github.com/edx/open-edx-proposals/pull/53#discussion_r169795700"},"pull_request":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53"}},"in_reply_to_id":168252944},"pull_request":{"url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53","id":167033085,"html_url":"https://github.com/edx/open-edx-proposals/pull/53","diff_url":"https://github.com/edx/open-edx-proposals/pull/53.diff","patch_url":"https://github.com/edx/open-edx-proposals/pull/53.patch","issue_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/53","number":53,"state":"open","locked":false,"title":"OEP-1: Update and simplify","user":{"login":"nasthagiri","id":6772826,"avatar_url":"https://avatars1.githubusercontent.com/u/6772826?v=4","gravatar_id":"","url":"https://api.github.com/users/nasthagiri","html_url":"https://github.com/nasthagiri","followers_url":"https://api.github.com/users/nasthagiri/followers","following_url":"https://api.github.com/users/nasthagiri/following{/other_user}","gists_url":"https://api.github.com/users/nasthagiri/gists{/gist_id}","starred_url":"https://api.github.com/users/nasthagiri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nasthagiri/subscriptions","organizations_url":"https://api.github.com/users/nasthagiri/orgs","repos_url":"https://api.github.com/users/nasthagiri/repos","events_url":"https://api.github.com/users/nasthagiri/events{/privacy}","received_events_url":"https://api.github.com/users/nasthagiri/received_events","type":"User","site_admin":false},"body":"The purpose of this PR is to update and simplify the OEP process defined in OEP-1.\r\n\r\n**Review Period**\r\n\r\nFebruary 05, 2018 -> February 20, 2018\r\n\r\n**Arbiter**\r\n\r\n@cpennington \r\n\r\n**Motivation**\r\n\r\nSimplify the process for creating/maintaining Open edX Proposals. Going forward, we want to encourage teams to write down and propose their architectural and system-wide decisions so we can align with best practices, processes, and system architectures.\r\n\r\nUpdate the process given recent changes in the org (i.e., the formation of an Architecture team in lieu of a chief architect).\r\n\r\n**Changes**\r\n* Simplify process\r\n  * Favor announcing on Slack over emailing edx-code.\r\n  * For Best Practice OEPs, favor updating rather than replacing.\r\n  * Reiterate option to have multiple authors to share the load.\r\n  * Add an explicit \"Review Period\" so process is finite and clear.\r\n  * Documentation readability\r\n    * Slight rearranging of sections, with further table of contents.\r\n    * Break down submission process in 5 clear steps.\r\n    * Fix a few typos with State transitions.\r\n* Replace edX Chief Architect with Architecture Team.\r\n* Append type and brief title to an OEP's file name.\r\n  * Rationale: Taking this pattern from [architecture decision records](https://www.thoughtworks.com/radar/techniques/lightweight-architecture-decision-records) ([example](https://github.com/npryce/adr-tools/tree/master/doc/adr))\r\n* Remove \"Product Enhancement\" proposal type.\r\n  * Rationale: Seems like a separate process from developer docs as the product team would want to weigh in on these decisions.\r\n* Remove support for Google Docs for discussion.\r\n  * Rationale: We are trying to consolidate our developer docs and want to move away from the exclusiveness and non-discoverability of Google docs.","created_at":"2018-02-04T22:11:06Z","updated_at":"2018-02-21T22:15:39Z","closed_at":null,"merged_at":null,"merge_commit_sha":"bb0cf31d352361647fa470247ba0deecef59fbe6","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[{"id":844232768,"url":"https://api.github.com/repos/edx/open-edx-proposals/labels/process","name":"process","color":"6ed326","default":false}],"milestone":null,"commits_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53/commits","review_comments_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53/comments","review_comment_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls/comments{/number}","comments_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/53/comments","statuses_url":"https://api.github.com/repos/edx/open-edx-proposals/statuses/0bf355d08a8d85e6b7adc61aa2f675cdfee65057","head":{"label":"edx:arch/simplify-oep1","ref":"arch/simplify-oep1","sha":"0bf355d08a8d85e6b7adc61aa2f675cdfee65057","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":55162195,"name":"open-edx-proposals","full_name":"edx/open-edx-proposals","owner":{"login":"edx","id":3179841,"avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/edx/open-edx-proposals","description":"Proposals for Open edX architecture, best practices and processes ","fork":false,"url":"https://api.github.com/repos/edx/open-edx-proposals","forks_url":"https://api.github.com/repos/edx/open-edx-proposals/forks","keys_url":"https://api.github.com/repos/edx/open-edx-proposals/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/open-edx-proposals/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/open-edx-proposals/teams","hooks_url":"https://api.github.com/repos/edx/open-edx-proposals/hooks","issue_events_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/events{/number}","events_url":"https://api.github.com/repos/edx/open-edx-proposals/events","assignees_url":"https://api.github.com/repos/edx/open-edx-proposals/assignees{/user}","branches_url":"https://api.github.com/repos/edx/open-edx-proposals/branches{/branch}","tags_url":"https://api.github.com/repos/edx/open-edx-proposals/tags","blobs_url":"https://api.github.com/repos/edx/open-edx-proposals/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/open-edx-proposals/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/open-edx-proposals/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/open-edx-proposals/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/open-edx-proposals/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/open-edx-proposals/languages","stargazers_url":"https://api.github.com/repos/edx/open-edx-proposals/stargazers","contributors_url":"https://api.github.com/repos/edx/open-edx-proposals/contributors","subscribers_url":"https://api.github.com/repos/edx/open-edx-proposals/subscribers","subscription_url":"https://api.github.com/repos/edx/open-edx-proposals/subscription","commits_url":"https://api.github.com/repos/edx/open-edx-proposals/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/open-edx-proposals/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/open-edx-proposals/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/open-edx-proposals/contents/{+path}","compare_url":"https://api.github.com/repos/edx/open-edx-proposals/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/open-edx-proposals/merges","archive_url":"https://api.github.com/repos/edx/open-edx-proposals/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/open-edx-proposals/downloads","issues_url":"https://api.github.com/repos/edx/open-edx-proposals/issues{/number}","pulls_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/open-edx-proposals/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/open-edx-proposals/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/open-edx-proposals/labels{/name}","releases_url":"https://api.github.com/repos/edx/open-edx-proposals/releases{/id}","deployments_url":"https://api.github.com/repos/edx/open-edx-proposals/deployments","created_at":"2016-03-31T15:36:33Z","updated_at":"2017-05-25T07:55:50Z","pushed_at":"2018-02-14T04:42:40Z","git_url":"git://github.com/edx/open-edx-proposals.git","ssh_url":"git@github.com:edx/open-edx-proposals.git","clone_url":"https://github.com/edx/open-edx-proposals.git","svn_url":"https://github.com/edx/open-edx-proposals","homepage":"http://open-edx-proposals.readthedocs.io/","size":1504,"stargazers_count":5,"watchers_count":5,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":3,"license":null,"forks":6,"open_issues":3,"watchers":5,"default_branch":"master"}},"base":{"label":"edx:master","ref":"master","sha":"0de0804b0b26ac3dfb39561a5cec28004e87ee15","user":{"login":"edx","id":3179841,"avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"repo":{"id":55162195,"name":"open-edx-proposals","full_name":"edx/open-edx-proposals","owner":{"login":"edx","id":3179841,"avatar_url":"https://avatars3.githubusercontent.com/u/3179841?v=4","gravatar_id":"","url":"https://api.github.com/users/edx","html_url":"https://github.com/edx","followers_url":"https://api.github.com/users/edx/followers","following_url":"https://api.github.com/users/edx/following{/other_user}","gists_url":"https://api.github.com/users/edx/gists{/gist_id}","starred_url":"https://api.github.com/users/edx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edx/subscriptions","organizations_url":"https://api.github.com/users/edx/orgs","repos_url":"https://api.github.com/users/edx/repos","events_url":"https://api.github.com/users/edx/events{/privacy}","received_events_url":"https://api.github.com/users/edx/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/edx/open-edx-proposals","description":"Proposals for Open edX architecture, best practices and processes ","fork":false,"url":"https://api.github.com/repos/edx/open-edx-proposals","forks_url":"https://api.github.com/repos/edx/open-edx-proposals/forks","keys_url":"https://api.github.com/repos/edx/open-edx-proposals/keys{/key_id}","collaborators_url":"https://api.github.com/repos/edx/open-edx-proposals/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/edx/open-edx-proposals/teams","hooks_url":"https://api.github.com/repos/edx/open-edx-proposals/hooks","issue_events_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/events{/number}","events_url":"https://api.github.com/repos/edx/open-edx-proposals/events","assignees_url":"https://api.github.com/repos/edx/open-edx-proposals/assignees{/user}","branches_url":"https://api.github.com/repos/edx/open-edx-proposals/branches{/branch}","tags_url":"https://api.github.com/repos/edx/open-edx-proposals/tags","blobs_url":"https://api.github.com/repos/edx/open-edx-proposals/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/edx/open-edx-proposals/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/edx/open-edx-proposals/git/refs{/sha}","trees_url":"https://api.github.com/repos/edx/open-edx-proposals/git/trees{/sha}","statuses_url":"https://api.github.com/repos/edx/open-edx-proposals/statuses/{sha}","languages_url":"https://api.github.com/repos/edx/open-edx-proposals/languages","stargazers_url":"https://api.github.com/repos/edx/open-edx-proposals/stargazers","contributors_url":"https://api.github.com/repos/edx/open-edx-proposals/contributors","subscribers_url":"https://api.github.com/repos/edx/open-edx-proposals/subscribers","subscription_url":"https://api.github.com/repos/edx/open-edx-proposals/subscription","commits_url":"https://api.github.com/repos/edx/open-edx-proposals/commits{/sha}","git_commits_url":"https://api.github.com/repos/edx/open-edx-proposals/git/commits{/sha}","comments_url":"https://api.github.com/repos/edx/open-edx-proposals/comments{/number}","issue_comment_url":"https://api.github.com/repos/edx/open-edx-proposals/issues/comments{/number}","contents_url":"https://api.github.com/repos/edx/open-edx-proposals/contents/{+path}","compare_url":"https://api.github.com/repos/edx/open-edx-proposals/compare/{base}...{head}","merges_url":"https://api.github.com/repos/edx/open-edx-proposals/merges","archive_url":"https://api.github.com/repos/edx/open-edx-proposals/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/edx/open-edx-proposals/downloads","issues_url":"https://api.github.com/repos/edx/open-edx-proposals/issues{/number}","pulls_url":"https://api.github.com/repos/edx/open-edx-proposals/pulls{/number}","milestones_url":"https://api.github.com/repos/edx/open-edx-proposals/milestones{/number}","notifications_url":"https://api.github.com/repos/edx/open-edx-proposals/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/edx/open-edx-proposals/labels{/name}","releases_url":"https://api.github.com/repos/edx/open-edx-proposals/releases{/id}","deployments_url":"https://api.github.com/repos/edx/open-edx-proposals/deployments","created_at":"2016-03-31T15:36:33Z","updated_at":"2017-05-25T07:55:50Z","pushed_at":"2018-02-14T04:42:40Z","git_url":"git://github.com/edx/open-edx-proposals.git","ssh_url":"git@github.com:edx/open-edx-proposals.git","clone_url":"https://github.com/edx/open-edx-proposals.git","svn_url":"https://github.com/edx/open-edx-proposals","homepage":"http://open-edx-proposals.readthedocs.io/","size":1504,"stargazers_count":5,"watchers_count":5,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":6,"mirror_url":null,"archived":false,"open_issues_count":3,"license":null,"forks":6,"open_issues":3,"watchers":5,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53"},"html":{"href":"https://github.com/edx/open-edx-proposals/pull/53"},"issue":{"href":"https://api.github.com/repos/edx/open-edx-proposals/issues/53"},"comments":{"href":"https://api.github.com/repos/edx/open-edx-proposals/issues/53/comments"},"review_comments":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53/comments"},"review_comment":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/edx/open-edx-proposals/pulls/53/commits"},"statuses":{"href":"https://api.github.com/repos/edx/open-edx-proposals/statuses/0bf355d08a8d85e6b7adc61aa2f675cdfee65057"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"nasthagiri"}}