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

Added min and max filters #475

Closed
wants to merge 1 commit into from
Closed

Added min and max filters #475

wants to merge 1 commit into from

Conversation

snoack
Copy link
Contributor

@snoack snoack commented Aug 7, 2015

4 years ago I suggested to add min and max filters, along with some other new filters as part of #66. That pull request has been closed, after some parts of it got implemented independently, even though somebody already complained there about these filters having been left out.

Also I still think adding these particular filters would be useful, as I've seen quite some projects adding custom filters for these purpose. So it seems to be a quite common use case. Also having them built into jinja2, allows us to support the optional attribute and case_sensitive options/semantics used by other built-in filters.

@snoack snoack mentioned this pull request Aug 7, 2015
@davidism davidism mentioned this pull request Jul 6, 2017
@davidism
Copy link
Member

davidism commented Jul 6, 2017

Continued in #737

@davidism davidism closed this Jul 6, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants