On 10/8/12 11:09 AM, Kevin Brosnan wrote:
I agree with Gavin this makes reading a bug much simpler when it comes
to understanding where a patch has landed especially when backouts
occur. The information is added for other readers of the bug not the
developer of the patch.
I concur and dissent. ;)
The info is generally useful, especially in the current system. So I'd
encourage people to keep adding "Pushed to inbound: <changeset url>" or
some flavor thereof. It's helpful for determining what's
happened/happening to a bug, 10x when backouts are involved. I suspect
having the inbound changeset is useful for someone doing regression
hunting (ie, looking between merges)?
OTOH, there's room for improvement as more automation becomes involved,
particularly for the simple case of land-merge-done. If a single,
automated, post-merge comment just noted everything, that would be
peachy. As would a single "landed, oops, backed out" note.
tl;dr: seems like useful info, but am not too hung up on the details. :)
Justin
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
