Automatically accept/reject beta/stable nominations

[spastorino]

It would be cool if from the meeting we could say @triagebot beta-accept #1111 or @triagebot beta-reject #1111 and have triagebot adding a comment on github doing what needs to be done to accept/reject a backport.

[apiraino]

@spastorino I'm on this issue.

Can you detail a bit what you mean by "what needs to be done to accept/reject a backport"? I was thinking to apply either the label beta-accepted or beta-rejected.

Perhaps would you like something more like in this comment?

[spastorino]

Perhaps would you like something more like in this comment?

👍, we should just fire that comment that also does the labelling adjustments.

[LeSeulArtichaut]

Can you detail a bit what you mean by "what needs to be done to accept/reject a backport"? I was thinking to apply either the label beta-accepted or beta-rejected.

Btw there is no beta-rejected label. The procedure is "add beta-accepted (without removing beta-nominated) if accepted, or remove beta-nominated if rejected.
I think the detailed procedure can be found in the Forge

[LeSeulArtichaut]

Also does triagebot have a proper code structure to create Zulip commands more easily now?

[apiraino]

Btw there is no beta-rejected label. The procedure is "add beta-accepted (without removing beta-nominated) if accepted, or remove beta-nominated if rejected.

ah good point, didn't know that. Are you referring to this documentation? In my opinion, not as clear as you wrote it here. Worth a small PR to clarify the wording?

Also does triagebot have a proper code structure to create Zulip commands more easily now?

mmm are you referring perhaps to issue #1 ? I can't tell, don't know what the plan was. I'm still wrapping my head and figuring out the actual workflow and API to/from Zulip, so I may be off, but by reading the code I think that commands have their own handler and adding a new handler is some copy&paste.

[spastorino]

Btw there is no beta-rejected label. The procedure is "add beta-accepted (without removing beta-nominated) if accepted, or remove beta-nominated if rejected.

ah good point, didn't know that. Are you referring to this documentation? In my opinion, not as clear as you wrote it here. Worth a small PR to clarify the wording?

I think @LeSeulArtichaut refers to our wg procedure document, I think there it's clear or at least should :).

[camelid]

I believe this should be moved to rust-lang/triagebot – cc @Mark-Simulacrum

[LeSeulArtichaut]

I'd argue that it's still useful to at list keep an issue here. It's a todo list of the things we want to have. We could link this to an issue in rust-lang/triagebot maybe