Merge lp://staging/~abeaumont/bzr-upload/revid-directory into lp://staging/bzr-upload

Proposed by Alfredo Beaumont
Status: Merged
Merge reported by: Vincent Ladeuil
Merged at revision: not available
Proposed branch: lp://staging/~abeaumont/bzr-upload/revid-directory
Merge into: lp://staging/bzr-upload
To merge this branch: bzr merge lp://staging/~abeaumont/bzr-upload/revid-directory
To post a comment you must log in.
Revision history for this message
Martin Albisetti (beuno) wrote :

This looks good.
Would you mind writing a test for it?
bearing that in mind, it seems good to merge to me.

Revision history for this message
Alfredo Beaumont (abeaumont) wrote :

Sounds great.
Let me take a look at testsuite api, and I'll write a test ASAP.

54. By Alfredo Beaumont

Make the test suite pass, but not the nearly empty one :).

55. By Alfredo Beaumont

Fix testsuite to work properly with revid-directory feature

Revision history for this message
Alfredo Beaumont (abeaumont) wrote :

> This looks good.
> Would you mind writing a test for it?
> bearing that in mind, it seems good to merge to me.

I've not written a test for it yet, but in the process i've made the testsuite pass with revid-dir patch. Any comment on the code is welcome.

Updating diff...

An updated diff will be available in a few minutes. Reload to see the changes.

Subscribers

People subscribed via source and target branches

to status/vote changes: