Add Ray technical charter#60068
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces the technical charter for the Ray project, defining its governance structure. The changes are well-structured and add important clarity for the community. My review includes suggestions to improve conciseness, fix a typo in the charter, and enhance the maintainability of the list of project members.
|
|
||
| ## General Project Policies | ||
|
|
||
| Ray has been established as Ray a Series of LF Projects, LLC. Policies applicable to Ray and participants in Ray, including guidelines on the usage of trademarks, are located at https://www.lfprojects.org/policies/. |
There was a problem hiding this comment.
There appears to be a typo here. "as Ray a" should probably be "as a".
| Ray has been established as Ray a Series of LF Projects, LLC. Policies applicable to Ray and participants in Ray, including guidelines on the usage of trademarks, are located at https://www.lfprojects.org/policies/. | |
| Ray has been established as a Series of LF Projects, LLC. Policies applicable to Ray and participants in Ray, including guidelines on the usage of trademarks, are located at https://www.lfprojects.org/policies/. |
There was a problem hiding this comment.
Intentional I believe
|
|
||
| ## Committers | ||
|
|
||
| Committers are responsible for approving pull requests before they are merged. Approval from a committer is required before a pull request can be merged. |
There was a problem hiding this comment.
The two sentences here are redundant. Consider rephrasing for conciseness and clarity.
| Committers are responsible for approving pull requests before they are merged. Approval from a committer is required before a pull request can be merged. | |
| Pull requests must be approved by a committer before being merged. |
| people | ||
| ``` | ||
|
|
||
| ## Summary |
There was a problem hiding this comment.
probably mention that it's part of LF/PTF here
6b17732 to
f629ad8
Compare
|
I built the docs locally and checked the formatting and such |
doc/source/ray-governance/index.md
Outdated
|
|
||
| ## Amendments | ||
|
|
||
| Charter amendments require (a) a supermajority (greater than 67%) of the TSC, (b) a majority (greater than 50%) of the lead maintainers, and (c) approval of LF Projects, LLC. |
There was a problem hiding this comment.
The (c) is formatted incorrectly at the moment, I can try to fix it.
Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com>
Signed-off-by: Philipp Moritz <pcmoritz@gmail.com>
Opening draft PR for Ray technical charter. Planning to add GitHub usernames before merging. --------- Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com> Co-authored-by: Philipp Moritz <pcmoritz@gmail.com>
Opening draft PR for Ray technical charter. Planning to add GitHub usernames before merging. --------- Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com> Co-authored-by: Philipp Moritz <pcmoritz@gmail.com> Signed-off-by: jeffery4011 <jefferyshen1015@gmail.com>
Opening draft PR for Ray technical charter. Planning to add GitHub usernames before merging. --------- Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com> Co-authored-by: Philipp Moritz <pcmoritz@gmail.com>
Opening draft PR for Ray technical charter. Planning to add GitHub usernames before merging. --------- Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com> Co-authored-by: Philipp Moritz <pcmoritz@gmail.com> Signed-off-by: peterxcli <peterxcli@gmail.com>
Opening draft PR for Ray technical charter. Planning to add GitHub usernames before merging. --------- Signed-off-by: Robert Nishihara <rkn@anyscale.com> Signed-off-by: Philipp Moritz <pcmoritz@gmail.com> Co-authored-by: Philipp Moritz <pcmoritz@gmail.com> Signed-off-by: peterxcli <peterxcli@gmail.com>

Adding Ray technical charter and list of Ray committers.