Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add default filter #36

Merged
merged 1 commit into from
Feb 3, 2016
Merged

Add default filter #36

merged 1 commit into from
Feb 3, 2016

Conversation

pathawks
Copy link
Member

It occurred to me that using the default filter in Liquid could allow us to dramatically simplify the logic of something like #33

Unfortunately, the default filter is not available in Jekyll 2, so I have provided it for compatibility.

I am sure that I am not namespacing properly. Filters needs to be a module somewhere, I am just not exactly sure where to put it.

@parkr
Copy link
Member

parkr commented Jan 29, 2016

👍

@pathawks pathawks self-assigned this Jan 30, 2016
@benbalter
Copy link
Collaborator

Nice. So much simpler. Thanks again, @pathawks!

benbalter added a commit that referenced this pull request Feb 3, 2016
@benbalter benbalter merged commit d73dc77 into master Feb 3, 2016
@benbalter benbalter deleted the pr/default branch February 3, 2016 17:32
CGarces added a commit to CGarces/CGarces.github.io that referenced this pull request Feb 3, 2016
Updated with last improvements of @benbalter
jekyll/jekyll-seo-tag#36
@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants