[RFC] Proposal to change handing of draft PRs by bot

[iarspider]

As discussed during yesterday's Core Software meeting, we propose to change how draft PRs are handled by cms bot:

• When a PR is in draft state, bot will not mention (@username) anyone in any of it's messages, except "New category assigned" ones
• It will also delay posting the "greeting" message ("A new Pull Request was created...") until the PR is marked as "Ready for review" for the first time

The commands (assign, unassign, test, etc) will still work as usual, and if necessary users can ping the corresponding experts themselves.

[cmsbuild]

cms-bot internal usage

[cmsbuild]

A new Issue was created by @iarspider.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[smuzaffar]

sounds good @iarspider
By the way, what about the PR updated message (#46650 (comment) ) . Are those be delayed too or should bot ping the explicitly assigned groups ?

[smuzaffar]

type doc

[iarspider]

By the way, what about the PR updated message (#46650 (comment) ) . Are those be delayed too or should bot ping the explicitly assigned groups ?

I think we can either skip them, or post a reduced version ("Pull request ... was updated", end of message) just to acknowledge that bot processed the new commit. All the assigned experts will get notified anyway in "greeting" message once it's posted.

[smuzaffar]

reduced version of the message sounds reasonable

[makortel]

Plan sounds good to me. Should we ping others (e.g. @cms-sw/all-l2) yet?

[makortel]

assign core, orp

[cmsbuild]

New categories assigned: core,orp

@antoniovilela,@Dr15Jones,@makortel,@mandrenguyen,@rappoccio,@sextonkennedy,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks

[iarspider]

@cms-sw/all-l2 any objections? Suggestions?

[fwyzard]

OK for me, I guess.

[smorovic]

No objections.

[smuzaffar]

this has been implemented and deployed via cms-sw/cms-bot#2395