Stream: t-release/triage

Topic: 2021-01-13 meeting


view this post on Zulip DPC (Jan 13 2021 at 16:00):

@WG-triage meeting time

view this post on Zulip DPC (Jan 13 2021 at 16:02):

Will wait for a few mins

view this post on Zulip DPC (Jan 13 2021 at 16:04):

So I guess its just the 2 of us :big_smile:

view this post on Zulip DPC (Jan 13 2021 at 16:04):

Happy new year and thanks for the efforts.

view this post on Zulip DPC (Jan 13 2021 at 16:06):

Not much to discuss today. Given the break, the number of prs to be triaged have increased

view this post on Zulip DPC (Jan 13 2021 at 16:07):

Though you don't have to triage all, you can do as much as possible and leave the rest for the next person to balance it out

view this post on Zulip DPC (Jan 13 2021 at 16:10):

Also ensure to skip triage only for members of teams not working groups or other smaller groups

view this post on Zulip DPC (Jan 13 2021 at 16:10):

Anyone has anything to bring up?

view this post on Zulip John Simon (Jan 13 2021 at 16:10):

Yes, it was about three hours of effort to catch everything up to about our usual fifteen days.

view this post on Zulip Alex Macleod (Jan 13 2021 at 16:11):

Nothing from me

view this post on Zulip DPC (Jan 13 2021 at 16:13):

Damn. You can do it in batches next week so it balances out with the others

view this post on Zulip DPC (Jan 13 2021 at 16:14):

Also don't adviss authors to ping the reviewer once ready as it might generate too many pings

view this post on Zulip DPC (Jan 13 2021 at 16:15):

If you wish you can instruct them to change labels

view this post on Zulip DPC (Jan 13 2021 at 16:16):

Using triagebot

view this post on Zulip John Simon (Jan 13 2021 at 16:16):

Happy to do it, though :)
should I leave the fact that I skipped doing triage for a PR because it was from a Rust team member out of the report?
also - is there an easier way to do the remove s-waiting-on-author add s-waiting-on-author process without just clicking on it?

view this post on Zulip DPC (Jan 13 2021 at 16:16):

No. It's k if it's in the report

view this post on Zulip Joshua Nelson (Jan 13 2021 at 16:16):

@John Simon there's a l keyboard shortcut

view this post on Zulip DPC (Jan 13 2021 at 16:17):

That way for some of them I can push them forward

view this post on Zulip Joshua Nelson (Jan 13 2021 at 16:17):

I don't think it can be fully automated though

view this post on Zulip Joshua Nelson (Jan 13 2021 at 16:17):

(and I kind of like that, it means you have to put a little thought into it)

view this post on Zulip DPC (Jan 13 2021 at 16:17):

You could use github's auto replies and put triagebot command in an auto reply

view this post on Zulip John Simon (Jan 13 2021 at 16:18):

What should I tell the PR authors to do in order to change labels?

view this post on Zulip DPC (Jan 13 2021 at 16:18):

That once ready for review they can use triagebot to change the labels to waiting on review

view this post on Zulip DPC (Jan 13 2021 at 16:19):

It's already there in the conflict message though

view this post on Zulip DPC (Jan 13 2021 at 16:20):

Anything else?

view this post on Zulip John Simon (Jan 13 2021 at 16:20):

nothing else... i see it -

bors commented on Nov 18, 2020
☔ The latest upstream changes (presumably #79167) made this pull request unmergeable. Please resolve the merge conflicts.
...
@rustbot modify labels: +S-waiting-on-review -S-waiting-on-author

view this post on Zulip DPC (Jan 13 2021 at 16:21):

Yeah

view this post on Zulip DPC (Jan 13 2021 at 16:21):

That should be it for today's meeting

view this post on Zulip DPC (Jan 13 2021 at 16:22):

See you next on 3rd February at the same time

view this post on Zulip DPC (Jan 13 2021 at 16:22):

Thanks and have a safe January ahead

view this post on Zulip Noah Lev (Jan 13 2021 at 20:36):

DPC said:

You could use github's auto replies and put triagebot command in an auto reply

Unfortunately I don't that will work for things like @rustbot label: +S-waiting-on-review -S-waiting-on-review because IIRC triagebot makes a diff and won't do no-op label changes. Also it generates a notification for everyone subscribed to the issue. I do wish there were an easier way than opening the label picker, removing the label, closing the picker, opening the picker, adding the label back, and closing it again :sweat_smile:

view this post on Zulip DPC (Jan 13 2021 at 22:57):

I wish so as well

view this post on Zulip Noah Lev (Jan 15 2021 at 04:47):

(deleted - wrong thread!)


Last updated: Jan 26 2022 at 14:02 UTC