IRC commit-bot: Make svn revision appear in the beginning of bot message.

IRC commit-bot: Make svn revision appear in the beginning of bot message.

It is easier to read the svn version quickly if it appears in the beginning of the message.

R=alancutter, dstockwell

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=166120

[vivekg, 2014-01-29 04:00:35]

PTAL

[vivekg, 2014-01-30 03:31:05]

ping!

[dstockwell, 2014-01-30 04:42:06]

I agree that it makes it easier to read the revision, but I find the revision to
be the least useful part of the message.

[vivekg, 2014-01-30 05:05:51]

On 2014/01/30 04:42:06, dstockwell wrote:
> I agree that it makes it easier to read the revision, but I find the revision
to
> be the least useful part of the message.

It may be useful to quickly grab the revision and its associated patch from the
IRC chat window while getting branch details from [1]

[1] http://omahaproxy.appspot.com/

[alancutter (OOO until 2018), 2014-01-30 06:13:21]

On 2014/01/30 05:05:51, vivekg_ wrote:
> On 2014/01/30 04:42:06, dstockwell wrote:
> > I agree that it makes it easier to read the revision, but I find the
revision
> to
> > be the least useful part of the message.
> 
> It may be useful to quickly grab the revision and its associated patch from
the
> IRC chat window while getting branch details from [1]
> 
> [1] http://omahaproxy.appspot.com/

This change seems reasonable to me. It makes sense to place strings of constant
width before varying width strings.
You might even consider putting the crrev.com link before the email address as
well.

[vivekg, 2014-01-30 06:39:01]

On 2014/01/30 06:13:21, alancutter wrote:
> On 2014/01/30 05:05:51, vivekg_ wrote:
> > On 2014/01/30 04:42:06, dstockwell wrote:
> > > I agree that it makes it easier to read the revision, but I find the
> revision
> > to
> > > be the least useful part of the message.
> > 
> > It may be useful to quickly grab the revision and its associated patch from
> the
> > IRC chat window while getting branch details from [1]
> > 
> > [1] http://omahaproxy.appspot.com/
> 
> This change seems reasonable to me. It makes sense to place strings of
constant
> width before varying width strings.
> You might even consider putting the http://crrev.com link before the email
address as
> well.

Thank you. Done the changes and uploaded a new patch. PTAL.

[alancutter (OOO until 2018), 2014-01-30 09:39:33]

On 2014/01/30 06:39:01, vivekg_ wrote:
> On 2014/01/30 06:13:21, alancutter wrote:
> > On 2014/01/30 05:05:51, vivekg_ wrote:
> > > On 2014/01/30 04:42:06, dstockwell wrote:
> > > > I agree that it makes it easier to read the revision, but I find the
> > revision
> > > to
> > > > be the least useful part of the message.
> > > 
> > > It may be useful to quickly grab the revision and its associated patch
from
> > the
> > > IRC chat window while getting branch details from [1]
> > > 
> > > [1] http://omahaproxy.appspot.com/
> > 
> > This change seems reasonable to me. It makes sense to place strings of
> constant
> > width before varying width strings.
> > You might even consider putting the http://crrev.com link before the email
> address as
> > well.
> 
> Thank you. Done the changes and uploaded a new patch. PTAL.

lgtm

[commit-bot: I haz the power, 2014-01-30 09:50:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vivek.vg@samsung.com/145103003/20001

[commit-bot: I haz the power, 2014-01-30 11:11:35]

Change committed as 166120

[commit-bot: I haz the power, 2014-01-30 11:12:14]

CQ bit was unchecked on CL. Ignoring.

[alancutter (OOO until 2018), 2014-01-30 21:44:46]

Running commit-bot instance updated to include this change.