Break djconvert into multiple packages #63

Closed
campadrenalin opened this Issue Sep 30, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@campadrenalin
Member

campadrenalin commented Sep 30, 2014

This is dependent on #60. Having an untestable, but tiny, main package, with the bulk of logic in an 'app' package, is a pattern that has worked very well for DJDNS. We should do the same thing here.

This may also be a good time to rethink how our Travis coverage testing command works. I think the line in DJDNS currently is superior.

@campadrenalin

This comment has been minimized.

Show comment
Hide comment
@campadrenalin

campadrenalin Oct 1, 2014

Member

We should be able to get multiple app package files, full coverage testing, various code simplifications, and the removal of the bash test suite from this ticket.

In case anyone thought the scope was tiny 😄

Member

campadrenalin commented Oct 1, 2014

We should be able to get multiple app package files, full coverage testing, various code simplifications, and the removal of the bash test suite from this ticket.

In case anyone thought the scope was tiny 😄

campadrenalin added a commit that referenced this issue Oct 2, 2014

campadrenalin added a commit that referenced this issue Oct 3, 2014

campadrenalin added a commit that referenced this issue Oct 3, 2014

campadrenalin added a commit that referenced this issue Oct 5, 2014

campadrenalin added a commit that referenced this issue Oct 5, 2014

campadrenalin added a commit that referenced this issue Oct 5, 2014

campadrenalin added a commit that referenced this issue Oct 5, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment