search

Swaagie / minimize

Minimize HTML
MIT License
162 stars 18 forks source link

SSI check other tags #82

Closed humby12 closed 7 years ago

humby12 commented 7 years ago

Previuous lib checked only include SSI tags, now with this condition, others SSI tags will be checked (if expr, else, endif, setvar echo var ...)

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 97.561% when pulling edbca8b02f155713b363555b7a96c6a5a99ab9c6 on humby12:patch-2 into 62092133d57fbf1e5cca72d3331a647a222e91f9 on Swaagie:master.

chiefcll commented 7 years ago

Whats the status of getting this merged? Using SSI Commands is important. I've added another patch which will do the same thing with tests and is slightly simpler. Anything with <!--# is considered a command and left intact.

Swaagie commented 7 years ago

Sorry for the delay, being a parent can be rough on time ;) Slowly getting back into OS groove. Will close this in favor of the other PR #83 (which is merged now). Thank you for your contribution