Skip to content

fix: update Gemfile.lock to include jekyll-sitemap#61

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

fix: update Gemfile.lock to include jekyll-sitemap#61
OriNachum merged 1 commit into
mainfrom
fix/update-gemfile-lock

Conversation

@OriNachum

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • CI pages build passes (no more bundle install failure)

  • Sitemap is generated at /sitemap.xml

  • Claude

The jekyll-sitemap gem was added to Gemfile but the lockfile was not
regenerated, causing CI bundle install to fail with exit code 16.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 8, 2026 21:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Update Gemfile.lock to include jekyll-sitemap dependency

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Regenerates Gemfile.lock to include jekyll-sitemap (v1.4.0) which was added to Gemfile in a
  previous PR but the lockfile wasn't updated
• Fixes CI failure caused by bundle install exit code 16
• Ensures sitemap is properly generated at /sitemap.xml
Diagram
flowchart LR
  A["Gemfile<br/>with jekyll-sitemap"] -- "regenerate" --> B["Gemfile.lock<br/>updated"]
  B -- "resolves" --> C["CI bundle install<br/>failure fixed"]
  C -- "enables" --> D["Sitemap generation<br/>at /sitemap.xml"]
Loading

Grey Divider

File Changes

Grey Divider

Qodo Logo

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

No Changes in PR

Qodo reviewed your PR and found no changes in the code

Grey Divider

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

Grey Divider

Qodo Logo

@sonarqubecloud

sonarqubecloud Bot commented Apr 8, 2026

Copy link
Copy Markdown

@OriNachum OriNachum merged commit 037c3a9 into main Apr 8, 2026
11 checks passed
@OriNachum OriNachum deleted the fix/update-gemfile-lock branch April 8, 2026 21:58
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