redding/sanford — pull request #122 by jcredding (2014-06-24 03:02:24+00:00)
Title: Remove `Runner.run` and `ServiceHandler.run`

This removes the `Runner.run` and the `ServiceHandler.run` class
methods. These were a neat feature but haven't really worked out
or been used. Since these haven't been used, this removes them so
they no longer have to be maintained. With planned changes to the
`Runner.new` args, this avoids having to update both of these
methods to using the new args.

@kellyredding - Ready for review. I'm working through changing the `Runner.new` to take a server's config data as its third arg and these methods were annoying to update. I didn't want to change them to expect a server's config data to be passed to them, so either I have to build a config data in the methods and then pass it to the sanford runner or change the runner logic to conditionally build a config data if one isn't passed. Since we haven't had any use for these methods, I thought it would be better to remove them instead of taking the time to get them to work.