Definitely. Will do. 

Will probably delete this pull request and make a new one (so as not to have a bunch of garbage commits)..

Sent from my iPhone

On Feb 3, 2013, at 6:04 AM, Jon Leighton <notifications@github.com> wrote:

> In lib/spring/commands.rb:
> 
> > @@ -124,5 +124,15 @@ def call(args)
> >        end
> >      end
> >      Spring.register_command "runner", Runner.new, alias: "r"
> > +
> > +    class Stop
> > +      def call(args)
> > +        env = Spring::Env.new
> > +        if env.pidfile_path.exist? && pid = env.pidfile_path.read
> > +          Process.kill('SIGTERM', pid.to_i)
> > +        end
> > +      end
> > +    end
> > +    Spring.register_command "stop", Stop.new
> There are now two types of "command" in spring: "application commands" (e.g. rake, test, console) and "client commands" (e.g. help, binstub). This file contains the application commands. But stop should be a client command.
> 
> You can see examples of client commands in lib/spring/client. You should probably extract the server_running? logic from the Run command into the superclass and reuse it in your Stop class. Make sense?
> 
> —
> Reply to this email directly or view it on GitHub.
>