GitHub

[David Golden]

On Tue, Jun 2, 2009 at 11:43 AM, Andy Armstrong <andy at hexten.net> wrote:
> http://github.com/AndyA/Test--Harness--do-not-use-me-please/tree/master
>
> I'd be grateful if interested parties could take a look at it and
> decide whether it looks sensible.

Immediate reactions:

(1) please, please, don't use double-dashes in the name, that's very
anti-convention and will be constantly mistyped.

(2) branches -- you seem to have branches for each project that point
to the same commit. Those should be eliminated. Likewise, you have
some branches that appear to have been deleted in the past and have
been resurrected.  See "git branch -av" for list of branches plus last
commit id/message.

More generally -- if these branches are all legacy and have been
merged into trunk or abandoned, you might just leave them out of the
conversion entirely.  I'd only keep branches if you expect future
commits on the branch.

(3) tags -- it looks like you've got 'tft-0.1', etc. for subproject
tags.  I would suggest following the same format for Test-Harness
commits or possibly expanding them all out fully:
"Test-More-Diagnostic-0.2", "Test-Harness-3.17", etc.  That will
minimize any chance of confusion over what the tags represent.

-- David