Mike Kupfer <mike.kupfer at sun.com> writes:

> I just finished going through the P1-P3 bugs and doing some simple risk
> analysis (likelihood of introducing breakage, possibility of being
> harder than the initial estimate).
>
> I added 456 ("DbLookups nits") to the blocker list for the internal
> review.  This was less about risk and more about fixing known problems
> that we'd just flag again in our internal review.
>
> The only other bugs that caught my eye were all Cadmium things:
>
>   356 cdm recommit needs to cope with the tags it may be trashing
>   418 git formatted hg diffs can lose copies/renames
>   464 ActiveList build doesn't consider one half of a working copy merge.
>
> Rich has a code review request out for 356, and he's asked me to review
> 464.
>
> Does 418 still need to be a stopper?  Or can we just document it as a
> restriction?

I think it could be documented as a restriction, but other opinion is desired.

> Also, I'd been keeping Cadmium separate from the rest of the tools in
> terms of planning for reviews.  Do we want to keep doing that?  The list
> of open issues for Cadmium is small enough that maybe we should plan on
> reviewing it at the same time as everything else...?

Sounds good to me, the more people who look at it, the better.

-- Rich

Reply via email to