modcloth/docker-builder — pull request #108 by rafecolton (2014-08-04 05:24:46+00:00)
Title: Bobconfig bobfile and container groups

This pull request has quite a few changes, as I went on a bit of a refactoring spree. Here's a list:

* minor
- accept path to `Bobfile` as one of the parameters in http invocations of `docker-builder`
* includes sanitization of the `Bobfile` path to avoid malicious `Bobfile` paths
* ensuring that we accept a workdir with symlinks just as long as the relative path from the repo top to the `Bobfile` does not contain any
- implement custom error types for both the builder and the parser

* patch
- rename top level directories not containing go source files to begin with a capital letter
- update many dependencies
- replace calls to `fmt.Sprintf` with `+` when all arguments are strings (better performance)
- making sure we print error messages with async builds
- refactor of the parser and lint functinality
* dramatically simplifying things
* no longer opening the `Bobfile` an extra, unnecessary time
- cleaning up some relics remaining from goworker

Addresses the following issues:

* https://github.com/modcloth/docker-builder/issues/104 (partial)
* https://github.com/modcloth/docker-builder/issues/98
* https://github.com/modcloth/docker-builder/issues/96 (just the `Bobfile` path part)