Skip to content

fix: add Jekyll _config.yml to stop Pages build failing on Liquid syntax - #160

Merged
jackgranatowski merged 2 commits into
mainfrom
claude/wcag-checker-undo-batch
May 30, 2026
Merged

fix: add Jekyll _config.yml to stop Pages build failing on Liquid syntax#160
jackgranatowski merged 2 commits into
mainfrom
claude/wcag-checker-undo-batch

Conversation

@jackgranatowski

@jackgranatowski jackgranatowski commented May 30, 2026

Copy link
Copy Markdown
Contributor

$(cat <<'EOF'

Summary

  • Adds a minimal _config.yml at the repo root with render_with_liquid: false
  • Fixes the red ✗ on main caused by the GitHub Pages Jekyll build failing

Root cause

docs/rebemer.md contains JSDoc type annotations using {{...}} syntax inside a fenced code block (lines 400–448). Jekyll's Liquid parser evaluates {{ }} even inside code fences, producing a build error. This was introduced when the reBEMer docs were added (before PR #159).

Fix

render_with_liquid: false in _config.yml disables Liquid processing site-wide. This is correct — the docs site has no Liquid templates and shouldn't need them.

Test plan

  • GitHub Pages "Build with Jekyll" check turns green on main after merge

https://claude.ai/code/session_018u9uVFxgL7K6EpPjZPggKR
EOF
)


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Updated configuration settings for template rendering behavior.

Review Change Stack

docs/rebemer.md contains {{...}} JSDoc type annotations inside a code
fence that Jekyll's Liquid parser tries to render, failing the GitHub
Pages build. render_with_liquid: false disables Liquid processing
site-wide, which is correct for a static docs site with no templates.

https://claude.ai/code/session_018u9uVFxgL7K6EpPjZPggKR
@coderabbitai

coderabbitai Bot commented May 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@jackgranatowski, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 57 minutes and 54 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 11ceb638-f9be-4432-9476-8a817f376003

📥 Commits

Reviewing files that changed from the base of the PR and between af467f2 and 5d0acf1.

📒 Files selected for processing (1)
  • _config.yml
📝 Walkthrough

Walkthrough

This PR disables Liquid template rendering in the Jekyll site configuration by setting render_with_liquid to false in _config.yml. This single-line configuration change prevents Jekyll from processing Liquid template syntax when building the site.

Changes

Jekyll Configuration Update

Layer / File(s) Summary
Liquid rendering setting
_config.yml
render_with_liquid is set to false to disable Liquid template processing during Jekyll site builds.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: adding a Jekyll configuration to prevent GitHub Pages build failures caused by Liquid syntax parsing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/wcag-checker-undo-batch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@_config.yml`:
- Line 1: The global render_with_liquid: false setting in _config.yml is
invalid; remove it from the top-level and either add render_with_liquid: false
to the YAML front matter of docs/rebemer.md or configure a Front Matter Defaults
entry in _config.yml (use the defaults key with scope:path set to "docs" or to
"docs/rebemer.md" and values:{render_with_liquid: false}) so the Liquid renderer
is disabled for that file/path.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a4884ae7-80b9-4b45-bad6-41c5b6d06f19

📥 Commits

Reviewing files that changed from the base of the PR and between 015e785 and af467f2.

📒 Files selected for processing (1)
  • _config.yml

Comment thread _config.yml Outdated
render_with_liquid is a front-matter variable, not a valid top-level
_config.yml key. Using Front Matter Defaults with scope path 'docs'
is the correct way to disable Liquid rendering for the whole docs
directory.

https://claude.ai/code/session_018u9uVFxgL7K6EpPjZPggKR
@jackgranatowski
jackgranatowski merged commit 3e04f99 into main May 30, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants