Bug 107527 - sheriff-bot should have a cc-gardeners command
Summary: sheriff-bot should have a cc-gardeners command
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-22 01:57 PST by Eric Seidel (no email)
Modified: 2017-07-18 08:30 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2013-01-22 01:57:06 PST 
sheriff-bot should have a cc-gardeners command

Basically the only time I ever ask sheriff-bot for what gardeners there are is so that I can CC them on some bug that I'm about to land, that I think they might want to see go by.

I'm not sure how many other people do this, but it seems like "good practice" to me, so the gardener isn't surprised by your new feature which causes 50 test to fail on some obscure bot.

<eseidel> sheriff-bot: cc-gardeners 12345
<sheriff-bot> eseidel: Added chromium gardeners abarth and dpranke to bug 12345.
Comment 1 Eric Seidel (no email) 2013-01-22 01:58:34 PST 
If I were to implement this (which I might), I would make a hidden webkit-patch command (possibly even a Step and just make it a DeclarativeSequencedCommand or whatever the class is) called "cc-gardeners" and then just have SB wrap that.  I think many of the SB commands already work this way.

Then again, I might decide that's too much overhead and just write it in SB directly.