-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Create a toolchain team. #179
Conversation
Create a team to oversee the development of Carbon's toolchain. This team is responsible for code reviews as well as handling any toolchain-specific decisions.
Posting this for an RFC -- pretty minor proposal overall. |
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.
Basically LGTM, although thinking non-proposal documentation should be clearer
docs/project/groups.md
Outdated
@@ -61,3 +61,9 @@ For most purposes, the Core team should be contacted about conduct issues. | |||
- Discord Chat role: review-managers | |||
- [Google group](https://groups.google.com/g/carbon-lang-review-managers): | |||
Grants Google Drive access. | |||
|
|||
### Toolchain team |
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.
Should this file start having short descriptions of the teams?
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.
Probably. I've added one for the toolchain team to the proposal.
We should also probably rotate the "admins" into an "infrastructure team" or something similar, but I figure that can happen later if/when relevant.
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.
Thanks. Updated a bit to expand on the scope of this and implications. I'll also ask for a review manager and hopefully set a comment deadline ASAP (given holidays and such).
docs/project/groups.md
Outdated
@@ -61,3 +61,9 @@ For most purposes, the Core team should be contacted about conduct issues. | |||
- Discord Chat role: review-managers | |||
- [Google group](https://groups.google.com/g/carbon-lang-review-managers): | |||
Grants Google Drive access. | |||
|
|||
### Toolchain team |
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.
Probably. I've added one for the toolchain team to the proposal.
We should also probably rotate the "admins" into an "infrastructure team" or something similar, but I figure that can happen later if/when relevant.
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.
I'm realizing I'm mostly commenting on the proposal edits. Take them or leave them, I had been thinking that was the actual doc.
Co-authored-by: Jon Meow <[email protected]> Co-authored-by: austern <[email protected]> Co-authored-by: Geoff Romer <[email protected]>
While this is still under decision, essentially all the comments were completely trivial punctuation, spelling, and word choice fixes with no impact on the document as a whole so I've applied them. Sorry for all the typos. |
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.
Proposal has been accepted on 2020-12-08. Announcement thread.
Updated to call the team the "implementation team" in the actual project docs, but I've left the proposal pristine and as it was viewed by the core team. |
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.
Community comment period has ended. PR approved.
Create a team to oversee the implementation of Carbon. This team is responsible for code reviews as well as handling any implementation-specific decisions. Originally, it was suggested to call this a "toolchain" team. During discussion, it became clear that the scope wasn't easily limited to implementation efforts that were necessarily part of the toolchain. There are many different aspects of implementation, and we anticipate code sharing between them that make it unhelpful to try to separate these. Instead, this team is chartered with broadly covering all of the implementation concerns in Carbon. If someone is concerned that an issue is larger than that, perhaps impacting the design of the language, the project, or the community, the standard process already provides for easy escalation to the core team. Co-authored-by: Jon Meow <[email protected]> Co-authored-by: austern <[email protected]> Co-authored-by: Geoff Romer <[email protected]>
Create a team to oversee the development of Carbon's toolchain. This
team is responsible for code reviews as well as handling any
toolchain-specific decisions.