Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

moduleLoader pod #866

Closed
wants to merge 10 commits into from
Closed

moduleLoader pod #866

wants to merge 10 commits into from

Conversation

mokko
Copy link
Contributor

@mokko mokko commented Nov 25, 2012

noticed an error in ModuleLoader pod today.

Sorry cant get this pR right. Somethings wrong with my git setup. Will try another time unless somebody wants to cherry pick the few lines of real changes etc. Thanks!

Alexis Sukrieh and others added 10 commits November 25, 2012 11:42
Signed-off-by: Pedro Melo <melo@simplicidade.org>
Signed-off-by: Pedro Melo <melo@simplicidade.org>
more hook pod updates
…ocumentation

The Plack middlware interface expects an array of arrays, where
the first element of each one is the middleware name, and the rest
are the arguments to that middleware.  The Cookbook suggests a
name and array-reference is required.

This change adjusts the data structures used appropriately (both
in code and in YAML).

Tested against Dancer 1.3095 and Plack 0.9988
@xsawyerx
Copy link
Member

I'll cherry-pick it in a bit.

@mokko
Copy link
Contributor Author

mokko commented Nov 25, 2012

i pushed a new version. GH-867. Sorry for bad PR again

@mokko mokko closed this Nov 25, 2012
@mokko
Copy link
Contributor Author

mokko commented Nov 25, 2012

i pushed a new version. Sorry for confusion.

On Sun, Nov 25, 2012 at 12:06 PM, Sawyer X notifications@github.com wrote:

I'll cherry-pick it in a bit.


Reply to this email directly or view it on GitHubhttps://github.com/sukria/Dancer/pull/866#issuecomment-10695612.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants