bring docs up-to-date with actual wsgify signature #167

Merged
merged 1 commit into from Apr 4, 2015

Projects

None yet

2 participants

@mmerickel
Member

the tests and code clearly expect (req, app, *_kw) and it makes sense that it
would be that way since req is the ephemeral argument and the rest are
defined at setup-time. The documented order before was (app, req, *_kw)
which puts the req in-between multiple config arguments in the
signature.

fixes #25, #120

@mmerickel mmerickel bring docs up-to-date with actual wsgify signature
the tests and code clearly expect (req, app, **kw) and it makes sense that it
would be that way since req is the ephemeral argument and the rest are
defined at setup-time. The documented order before was (app, req, **kw)
which puts the req in-between multiple config arguments in the
signature.

fixes #25, #120
e4e8c47
@bertjwregeer
Member

👍 Looks good to me.

@bertjwregeer bertjwregeer added this to the Version 1.5 milestone Mar 23, 2015
@bertjwregeer bertjwregeer merged commit e4e8c47 into master Apr 4, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@bertjwregeer bertjwregeer deleted the fix.wsgify-middleware-docs branch Apr 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment