@mikelsr As warned, this ended up being a pretty large diff.
Changes include:
package restructuring
renaming of various symbols (not 100% convinced about the process.Server name; let me know what you think).
eschew process constructor for config/factory pattern in pkg/server
remove IO streams, as per today's discussion
change host code to return a non-zero exit code
add unit test
Let me know what you think!
P.S. It might be easier to checkout the cleanup/proc-lazy-init branch and just read that directly. The diff is super noisy (though much of it is file renames/deletes). If you decide to do this, running the unit test is probably a good place to start.
@mikelsr As warned, this ended up being a pretty large diff.
Changes include:
process.Servername; let me know what you think).pkg/serverLet me know what you think!
P.S. It might be easier to checkout the
cleanup/proc-lazy-initbranch and just read that directly. The diff is super noisy (though much of it is file renames/deletes). If you decide to do this, running the unit test is probably a good place to start.