Oliver Steele

Results 29 comments of Oliver Steele

The test suite shows [these errors](https://github.com/osteele/gojekyll/runs/5171787377?check_suite_focus=true). There appear to be three categories of errors: - `"\n"` vs. `"\r\n"`. gojekyll itself may be behaving correctly here; it might be that the...

Liquid now knows about MapSlice.

Until this issue is resolved, would it make sense to remove this text from ? It is so far as I can tell factually incorrect. If there is some sense...

I had expected csstree to preserve comments, based on the description “focus on […] source-to-source transforming tasks” at the top of the README. As an (appreciative) user of the library,...

PR #472 looks like it implements this. It's currently failing CI, and un-merged.

One fix might look something like the following. (I don't really know my way around the Jekyll code base, so there's doubtless issues with this.) 1. `Document.read_content` sets a new...

谢谢你的错误报告!我终于有时间来处理它了 Shopify Liquid does not support Chinese variable names either. Shopify/liquid#31 closes it as "won't fix". (It's actually got a richer history. It was fixed, and then the fix was...

Here's an analysis and what it would take to fix this: ## Current State ### This Implementation - The Ragel lexer in `expressions/scanner.rl` uses ASCII-only character classes - Pattern: `identifier...

Thanks @connordelacruz for the informative bug report, and thanks @drskullster for the workaround. I'll take a look.