Added decorators #296

Closed
wants to merge 4 commits into
from

Conversation

2 participants
@Poincare

Poincare commented Aug 5, 2011

I have added decorators for before_request and after_request, please ignore the README changes, they were errors I had trouble reversing. If it doesn't apply cleanly, please tell me so I can fix it.

Also, keep up the excellent work, I love flask.

@mitsuhiko

This comment has been minimized.

Show comment
Hide comment
@mitsuhiko

mitsuhiko Aug 5, 2011

Member

before_request can already be used as a decorator:

@app.before_request
def my_function():
    ...
Member

mitsuhiko commented Aug 5, 2011

before_request can already be used as a decorator:

@app.before_request
def my_function():
    ...
@mitsuhiko

This comment has been minimized.

Show comment
Hide comment
@mitsuhiko

mitsuhiko Aug 5, 2011

Member

Also, our whitespace rules are consistent: two spaces after fullstops in preformatted text.

Member

mitsuhiko commented Aug 5, 2011

Also, our whitespace rules are consistent: two spaces after fullstops in preformatted text.

@mitsuhiko mitsuhiko closed this Aug 5, 2011

@Poincare

This comment has been minimized.

Show comment
Hide comment
@Poincare

Poincare Aug 5, 2011

Ah, I see thanks for telling me.

Poincare commented Aug 5, 2011

Ah, I see thanks for telling me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment