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

Minify template just once #20

Merged
merged 1 commit into from
Jan 7, 2016
Merged

Minify template just once #20

merged 1 commit into from
Jan 7, 2016

Conversation

pathawks
Copy link
Member

@pathawks pathawks commented Jan 6, 2016

Instead of minifying the results each time the tag is rendered, minify the template when it is read from the file and never again.

@benbalter
Copy link
Collaborator

Huh. Never thought of that. Makes sense. 👍

@parkr
Copy link
Member

parkr commented Jan 6, 2016

🎉 :shipit:

pathawks added a commit that referenced this pull request Jan 7, 2016
@pathawks pathawks merged commit 860b8b9 into jekyll:master Jan 7, 2016
@pathawks pathawks deleted the minify-once branch January 7, 2016 02:32
@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