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 support for lstrip_blocks #152

Open
RosanneZe opened this issue Nov 11, 2019 · 4 comments
Open

add support for lstrip_blocks #152

RosanneZe opened this issue Nov 11, 2019 · 4 comments

Comments

@RosanneZe
Copy link

RosanneZe commented Nov 11, 2019

Would it be possible to add support for the lstrip_blocks setting?

@stevepiercy
Copy link
Member

Yes. Support for any Jinja2 configuration is possible. Pull requests to implement the feature request are welcome. See requirements for contributions for full details.

Briefly that includes:

@RosanneZe
Copy link
Author

I looked into it, and it is actually supported. It is not in the tests or the documentation, but it does work.

@stevepiercy
Copy link
Member

Please do share you findings with a pull request.

@RosanneZe
Copy link
Author

I didn't actually do any coding, just did a grep in the code. If you grep both lstrip_blocks and trim_blocks, you can see that they are both in the code, but only trim_blocks is in the tests. I thought it wasn't implemented, because it was not mentioned in the documentation, while trim_blocks was. (I checked that it worked by just adding it in my pyramid app and seeing if it made a difference).

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

No branches or pull requests

2 participants