xeps icon indicating copy to clipboard operation
xeps copied to clipboard

Facilitate triaging through automation

Open guusdk opened this issue 3 years ago • 2 comments

Triaging is a manual process. It would be great to have a summary of things to manually watch out for as a comment by a bot in the PR.

guusdk avatar Oct 13 '22 12:10 guusdk

@horazont (given that I've copied this suggestion from your email): would this be solved by a CI bot that would comment on every PR to this repository with a static text? If so, maybe a solution could be based on https://github.com/marketplace/actions/auto-comment

guusdk avatar Oct 13 '22 12:10 guusdk

That would be a start, yes.

Ideally combined with #1223.

horazont avatar Oct 15 '22 14:10 horazont

Thank you Georg for the support!

  • [ ] Estimated workload: TBD
  • [ ] Assumption: TBD
  • [x] Further clarification: What exactly is part of the triage? Is it simply about pre-build TODO-lists by the script?

Echolon avatar Oct 25 '22 16:10 Echolon

Basically, there should be a bot which implements all things which can be automated from https://github.com/xsf/xeps/blob/master/docs/TRIAGING.md#flowchart.

Example scenarios:

  • PR which adds a ProtoXEP to the inbox: the bot asks the Editor team to check for basic "publishability" (does not violate any XSF standard, e.g. by use obscene language)
  • PR which touches an Experimental XEP: the bot asks the Editor to either verify that the PR author is the author of the XEP (then it can be merged), or to ask the XEP author for confirmation
  • PR which touches a Stable XEP: the bot asks the Editor to check for editorial-only changes (can be merged) or to ask the approving body for confirmation.

Stuff like that. Just so that an Editor does not have to enter the diff view to find out which kind of further review.

horazont avatar Oct 25 '22 20:10 horazont

Closed in favour of #1237, #1240, which should make for a clearer work package. Re-open here if you disagree or comment over there if you have anything to add.

horazont avatar Nov 05 '22 09:11 horazont