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

Add logging to Undefined type. #339

Closed
wants to merge 1 commit into from

Conversation

@koddsson
Copy link

commented May 31, 2014

This is almost the same as pallets/flask#1028 although it does not check if we are running in debug mode.

Add logging to Undefined type.
This is almost the same as pallets/flask#1028 although it does not check if
we are running in debug mode.
@koddsson

This comment has been minimized.

Copy link
Author

commented May 31, 2014

Ah, I forgot to run the tests before making the PR. Trying to figure out how I can set and check if jinja is running in debug mode to make sure that we don't output to stdout all the time.

@mitsuhiko

This comment has been minimized.

Copy link
Member

commented Jun 6, 2014

I implemented this now through a factory function but not by changing the behavior of the default undefined type.

@mitsuhiko mitsuhiko closed this Jun 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.