Re: http://drupal.org/node/558534#comment-2264304

It says "deekayen requested that failed test be re-tested.", though I requested a re-test just to have a recent one. The previous test hadn't failed.

More generic alternatives:
"deekayen requested for a file to be re-tested."
"deekayen requested for a patch to be re-tested."
"deekayen requested a re-test."
"deekayen requested a new test."

CommentFileSizeAuthor
#2 632564-auto-followup-message.patch3.82 KBboombatower

Comments

boombatower’s picture

boombatower commented

From #631632: Change re-test comment format which was proposed in IRC by webchick and others (issue in wrong queue, by the maintain...lolz): Something like "Patch #111 snarfblatz.patch re-test requested by username".

which we later agreed made more sense as:
"#111 snarfblatz.patch re-test requested by username"

boombatower’s picture

boombatower commented
Title: Make re-test system message more generic » Rewrite automatic followup messages
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 632564-auto-followup-message.patch3.82 KB

The two messages:

Re-test of @filename from comment <a href="#@id">#@cid</a> was requested by @user.

and

The last submitted patch, <a href="#@id">@filename</a>, failed testing.
boombatower’s picture

boombatower commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.