Skip to content

Fix Gemfile.lock and sitemap plugin config#9

Merged
OriNachum merged 1 commit into
mainfrom
fix/update-gemfile-lock
Apr 8, 2026
Merged

Fix Gemfile.lock and sitemap plugin config#9
OriNachum merged 1 commit into
mainfrom
fix/update-gemfile-lock

Conversation

@OriNachum
Copy link
Copy Markdown
Contributor

Summary

  • Updates Gemfile.lock to include the jekyll-sitemap dependency
  • Fixes YAML indentation in _config.ymljekyll-sitemap was nested under jekyll-relative-links instead of being a sibling list item, causing the Jekyll build to fail

Test plan

  • bundle exec jekyll build succeeds

  • _site/sitemap.xml is generated

  • Claude
    🤖 Generated with Claude Code

The sitemap plugin entry was nested under jekyll-relative-links
instead of being a sibling list item, causing the Jekyll build to fail.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 8, 2026 22:05
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Fix jekyll-sitemap plugin YAML indentation in config

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Fixes YAML indentation in _config.yml for jekyll-sitemap plugin
• Corrects plugin nesting to be sibling list item, not nested under jekyll-relative-links
• Resolves Jekyll build failure caused by malformed plugin configuration
Diagram
flowchart LR
  A["Incorrect nesting<br/>jekyll-sitemap under<br/>jekyll-relative-links"] -- "Fix indentation" --> B["Correct sibling<br/>jekyll-sitemap as<br/>list item"]
Loading

Grey Divider

File Changes

1. _config.yml 🐞 Bug fix +1/-1

Fix jekyll-sitemap plugin indentation

• Corrected indentation of jekyll-sitemap plugin entry from 4 spaces to 2 spaces
• Changed jekyll-sitemap from nested item under jekyll-relative-links to sibling list item
• Aligns with proper YAML list formatting for plugins array

_config.yml


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Apr 8, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OriNachum OriNachum merged commit 4cba77e into main Apr 8, 2026
9 checks passed
@OriNachum OriNachum deleted the fix/update-gemfile-lock branch April 8, 2026 22:08
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