Oliver Steele

Results 29 comments of Oliver Steele

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...

> 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...

Thank you for your interest in contributing to this project! However, I'm unable to merge this PR as it appears to be a fork divergence rather than a focused contribution....

Blocked on #42. There is a PR under development for that issue; once it is merged I will fix this one.

The Windows CI is failing because tests use forward slashes in template cache keys but the `filepath.Join` creates backslash paths on Windows: open testdata\render_combined.html: The system cannot find the file...

Per the Shopify docs: when using `with` without `as`, the variable should be available by the snippet filename, not `"object"`: ``` {% render 'product-card' with product %} ``` > If...

## Investigation Results: Timezone Handling in Permalinks I've conducted empirical testing to compare how Jekyll and gojekyll handle timezones in permalink generation. **TL;DR: Gojekyll does not currently honor the `timezone`...

According to the [Kramdown syntax documentation](https://kramdown.gettalong.org/syntax.html), the markdown attribute values have the following meanings: - `0`: Disables markdown processing entirely - `1`: Enables markdown processing (default) - `block`: Parses the...

The code and tests already support `markdown="1"`. Kramdown also supports `markdown="0"`, while this gojekyll doesn't. I've expanded the title to address this.