Code review comment for lp:~vila/bzr/stack-remove-unknown

Vincent Ladeuil (vila) wrote :

> Seems fine.

Thanks.

> Does this want a separate test case, rather than just being seen
> in an absence of kipple on the existing tests?

I thought about it, but really, the warning caught that mistake and that should be enough.

Even the warning didn't really make sense and there is no unknown territory to explore with a new test.

« Back to merge proposal