Dep: Separate code into subpackage(s)

Created on 26 Jan 2017  路  5Comments  路  Source: golang/dep

To keep things moving quickly, our initial work on dep clustered everything into a single package. That's not ideal for the long term, though. I can imagine at least two discrete subsystems that need separating:

  • Manifest and Locks, plus the txn writer that works with them.
  • The analyzer. This is tiny right now, but may get a fair bit bigger over time, especially if we choose to teach it to read data from the existing community tools.

My first instinct is to put these in separate packages, but if we're trying to maintain consistency with how the toolchain does things, it might be better to just put them all in one, non-main subpackage.

enhancement help wanted

All 5 comments

It would be nice to work with lock and manifest files programmatically! Moving things out of the main package is something I am looking for.

Yes, it would be really nice to move everything out of the main package. If no one is on it I can work on it tomorrow. I'd only need to know if I should put everything in one non-main subpackage or go for the multi-package approach.

@erizocosmico awesome! i think just one is good to start with. I suspect it'll be as much updating the supporting/CI scripts as it is actually moving code around.

Nice, I'll do it asap.

178 is merged, so marking this done. We can make a new issue if we want to further subdivide the primary lib package later.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

angryrobot picture angryrobot  路  3Comments

jeffwillette picture jeffwillette  路  3Comments

michael-go picture michael-go  路  3Comments

cemremengu picture cemremengu  路  3Comments

abeMedia picture abeMedia  路  3Comments