librelist archives

« back to archive

PLUGINS.md

PLUGINS.md

From:
Chris Wanstrath
Date:
2010-04-02 @ 22:28
I just wrote a PLUGINS.md draft:

http://github.com/defunkt/resque/blob/master/docs/PLUGINS.md

These are standards for plugins released *after* Resque 2.0 is out. I
don't expect anyone to change their existing, perfectly functional gem
just because we finally decided to agree on some rules.

That said, what do you guys think? Anything that should be added / changed?

Chris

Re: [resque] PLUGINS.md

From:
Ryan Carver
Date:
2010-04-02 @ 22:45
Great start. Nothing else in particular comes to mind right now.

Though..

def test_lint
  assert Resque::Plugin.lint(Resque::Plugins::Lock)
end

`Resque::Plugin.lint` throws an exception when it finds a problem - the
return value isn't specified. Should it be?



On Fri, Apr 2, 2010 at 3:28 PM, Chris Wanstrath <chris@ozmm.org> wrote:

> I just wrote a PLUGINS.md draft:
>
> http://github.com/defunkt/resque/blob/master/docs/PLUGINS.md
>
> These are standards for plugins released *after* Resque 2.0 is out. I
> don't expect anyone to change their existing, perfectly functional gem
> just because we finally decided to agree on some rules.
>
> That said, what do you guys think? Anything that should be added / changed?
>
> Chris
>

Re: [resque] PLUGINS.md

From:
Chris Wanstrath
Date:
2010-04-02 @ 22:52
On Fri, Apr 2, 2010 at 3:45 PM, Ryan Carver <ryan@fivesevensix.com> wrote:

> Great start. Nothing else in particular comes to mind right now.
> Though..
> def test_lint
>   assert Resque::Plugin.lint(Resque::Plugins::Lock)
> end
> `Resque::Plugin.lint` throws an exception when it finds a problem - the
> return value isn't specified. Should it be?

Good point, fixed:

http://github.com/defunkt/resque/commit/4fcf1aa7d21cd0a27e8115eae32846428e2d03eb

Chris