Skip to content

Use Makefile in GH Action#526

Draft
RamVasuthevan wants to merge 5 commits intomainfrom
claude/explain-jekyll-workflow-kuRLz
Draft

Use Makefile in GH Action#526
RamVasuthevan wants to merge 5 commits intomainfrom
claude/explain-jekyll-workflow-kuRLz

Conversation

@RamVasuthevan
Copy link
Copy Markdown
Owner

No description provided.

Add a basic build target that runs Jekyll build without rbenv
dependency, making it suitable for CI/CD environments.
Add strict liquid configuration to _config.yml:
- error_mode: strict
- strict_variables: true
- strict_filters: true

Add --strict_front_matter flag to makefile build target.

This ensures builds fail on undefined variables, invalid filters,
and front matter parsing errors, making CI/CD more reliable.
- Rename workflow to "Jekyll Build (Strict Mode)"
- Update bundle install to use modern syntax (bundle config set)
- Simplify build step to use 'make build'
- Remove grep-based error detection (now handled by strict mode)

The workflow now relies on Jekyll's built-in strict mode to fail
on errors instead of fragile string matching.
- Change website/_.bundle/ to website/.bundle/ (fix typo)
- Change website/_vendor/ to website/vendor/ (correct path)

The previous patterns with underscore prefixes were not matching
the actual bundler-generated directories.
@RamVasuthevan RamVasuthevan force-pushed the claude/explain-jekyll-workflow-kuRLz branch from aee3050 to 89861d1 Compare January 11, 2026 18:28
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jan 11, 2026

Deploying personal-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: d3b069f
Status:🚫  Build failed.

View logs

Add nil checks for variables that may not be defined:
- all-pages.html: Check if page.sub_domains exists before iteration
- reading-list.html: Check if include.articles exists before iteration
- image.html: Add fallback for page.permalink (default to page.url)
- unsplash_image.html: Add fallback for page.permalink (default to page.url)

These fixes prevent errors when strict_variables is enabled in Jekyll.
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