liquid icon indicating copy to clipboard operation
liquid copied to clipboard
verified publisher icon osteele

Implemented auto-escape functionality and provided a default escape replacer for HTML.

Open dop251 opened this issue 6 months ago • 2 comments

Checklist

  • [x] I have read the contribution guidelines.
  • [x] make test passes.
  • [x] make lint passes.
  • [x] New and changed code is covered by tests.
  • [ ] Performance improvements include benchmarks.
  • [x] Changes match the documented (not just the implemented) behavior of Shopify.

dop251 avatar Jun 02 '25 14:06 dop251

Rebased version of https://github.com/osteele/liquid/pull/70

dop251 avatar Jun 02 '25 14:06 dop251

Fixed a small lint issue, should be fine now.

dop251 avatar Jun 02 '25 17:06 dop251

Thank you for your contribution!

Copilot found a couple of issues that appear to be valid. There's another PR waiting to land that might require re-merging this one, so I've gone ahead and merged it as is first. I'll create another PR to address the issues that Copilot found.

osteele avatar Sep 01 '25 05:09 osteele

Copilot found a couple of issues that appear to be valid.[...] I'll create another PR to address the issues that Copilot found.

Oops I accidentally pushed the commit to main instead of to a new branch. It's a71aedf199fcdbe30a92a87891e7bd11665ccc34 in case you want to take a look at it for a "post-merge" review.

osteele avatar Sep 01 '25 05:09 osteele