Skip to content
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

Retire the mailing list and make all decisions on zulip #649

Open
oli-obk opened this issue Jul 3, 2023 · 12 comments
Open

Retire the mailing list and make all decisions on zulip #649

oli-obk opened this issue Jul 3, 2023 · 12 comments
Labels
disposition-merge The FCP starter wants to merge this major-change A proposal to make a major change to rustc proposed-final-comment-period An FCP has been started, cast your votes and raise concerns T-compiler Add this label so rfcbot knows to poll the compiler team

Comments

@oli-obk
Copy link
Contributor

oli-obk commented Jul 3, 2023

Proposal

We already have a private zulip stream. I propose to use it for discussing things like moving a compiler contributor to the compiler team, or adding a new compiler contributor. Similar to how one can reply privately to compiler team leads on email, people should send private zulip DMs to them in case they don't want their comments visible to the entire compiler team and the compiler contributors.

Mentors or Reviewers

N/A

Process

This MCP will not get seconded, but FCPed. Unlike other FCPs, this one requires unanimous approval.

Comments

This issue is not meant to be used for technical discussion. There is a Zulip stream for that. Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.

@oli-obk oli-obk added T-compiler Add this label so rfcbot knows to poll the compiler team major-change A proposal to make a major change to rustc labels Jul 3, 2023
@rustbot
Copy link
Collaborator

rustbot commented Jul 3, 2023

This issue is not meant to be used for technical discussion. There is a Zulip stream for that. Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.

cc @rust-lang/compiler @rust-lang/compiler-contributors

@rustbot rustbot added the to-announce Announce this issue on triage meeting label Jul 3, 2023
@compiler-errors
Copy link
Member

@rfcbot fcp merge

@rfcbot
Copy link

rfcbot commented Jul 3, 2023

Team member @compiler-errors has proposed to merge this. The next step is review by the rest of the tagged team members:

Concerns:

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@rfcbot rfcbot added proposed-final-comment-period An FCP has been started, cast your votes and raise concerns disposition-merge The FCP starter wants to merge this labels Jul 3, 2023
@compiler-errors
Copy link
Member

Per comment above, I'll mark a concern that this is waiting on unanimous approval.

@rfcbot concern unanimous-approval

@compiler-errors
Copy link
Member

compiler-errors commented Jul 3, 2023

adding another concern:

We preferably want to have automatic sync between zulip and folks joining/leaving T-compiler{,-contributors} to make sure these things like membership conversations remain private and also we don't accidentally exclude folks. I've noted in the past that we're not super great at making sure this stays in sync.

@rfcbot concern automatic-sync

@apiraino apiraino removed the to-announce Announce this issue on triage meeting label Jul 6, 2023
@pnkfelix
Copy link
Member

pnkfelix commented Aug 3, 2023

@rfcbot concern single-point-of-failure-via-stream-archival

As I noted in Zulip, I'm a little nervous about the semi-destructive nature of Zulip's archive command. It might be good for us to have some backup mechanism for streams that are used to capture long-term decisions. (I feel email is fundamentally different here due to its distributed nature, in that we all sort of have a "shared backup", collectively...)

(I'm still torn about whether to treat this as a formal concern, i.e. something that should block progress here, or just as a warniing that all archive actions need to be done with care, at least until/unless zulip adds an "undo" for archive commands...)

@compiler-errors
Copy link
Member

@rfcbot resolve unanimous-approval

@apiraino
Copy link
Contributor

Closing MCP, probably stale. See MCP process.

If the topic of discussion in this MCP should be reignited, feel free to reopen this proposal or a new one.

@apiraino apiraino closed this as not planned Won't fix, can't repro, duplicate, stale May 16, 2024
@pnkfelix pnkfelix reopened this May 16, 2024
@rustbot
Copy link
Collaborator

rustbot commented May 16, 2024

This issue is not meant to be used for technical discussion. There is a Zulip stream for that. Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.

Concerns or objections to the proposal should be discussed on Zulip and formally registered here by adding a comment with the following syntax:

@rustbot concern reason-for-concern 
<description of the concern> 

Concerns can be lifted with:

@rustbot resolve reason-for-concern 

See documentation at https://forge.rust-lang.org

cc @rust-lang/compiler @rust-lang/compiler-contributors

@rustbot rustbot added the to-announce Announce this issue on triage meeting label May 16, 2024
@pnkfelix
Copy link
Member

(my intent in filing my concern was to raise awareness of a danger; I had not anticipated progress being stalled to the point where the MCP would just be closed. That was not be intent.)

I'm going to withdraw my concern.

@pnkfelix
Copy link
Member

@rustbot resolve single-point-of-failure-via-stream-archival

withdrawing my concern; I've tried to raise awareness of a potential problem. but I do not want that to actually cause this MCP to not occur.

@apiraino apiraino removed the to-announce Announce this issue on triage meeting label May 16, 2024
@apiraino
Copy link
Contributor

@rfcbot resolve single-point-of-failure-via-stream-archival

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
disposition-merge The FCP starter wants to merge this major-change A proposal to make a major change to rustc proposed-final-comment-period An FCP has been started, cast your votes and raise concerns T-compiler Add this label so rfcbot knows to poll the compiler team
Projects
None yet
Development

No branches or pull requests

6 participants