1.  I added a reference to RT 42360 which requests unit tests for the
SVK portion of Parrot::Revision's code.

2.  Would it be possible to add tests for the git-related code in your
patch to the following 4 test files:

t//configure/017-revision_no_DEVELOPING.t
t//configure/018-revision.t
t//postconfigure/03-revision_no_DEVELOPING.t
t//postconfigure/04-revision.t

In a part of the Parrot distribution which has long been stable, such as
this module, any code we add should be accompanied by tests so that we
maintain high test coverage and quality.  Perhaps if we get some tests
for a git block, we can shame some of our SVK users to write some tests
for the SVK block referenced in RT 42360.

Thank you very much.

kid51

Reply via email to