+1 Sounds lot of sense to me.  Maybe when commenting out it would be
good to include the number of the JIRA that has been opened for this.

- Venkat

On 7/17/07, Jean-Sebastien Delfino <[EMAIL PROTECTED]> wrote:
[snip]
Raymond Feng wrote:
> Hi,
>
> I checked the itest/pom.xml and saw quite a few modules are commented
> out. We removed the failing modules from the build "temporarily" but
> it ends that they are excluded from the build "permanently". We're
> cheating ourselves:-(
>
> IMHO, we need to handle these situations more carefully moving
> forward. I suggest that we follow the rules below:
>
> 1) Try to fix the failing test case if the break is caused by your
> changes before the check-in.
> Otherwise,
> 2) Open a JIRA to track the issue if you expect other people to fix
> it. Mark the JIRA as "blocker".
>
+1

I'd add:

3) Mark the JIRA resolved when the problem is fixed :)

--
Jean-Sebastien


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to