You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
docs: use bundle exec in the custom config example - #492
DEVELOPMENT.md told readers to run bundler exec jekyll serve for a custom config; the Bundler CLI command is bundle exec, matching the other examples in the file.
Docs-only change. Opened to exercise the webhook-fired review routine; do not merge until the review comment has appeared.
Request. No linked issue; the PR description asks to change the one custom-config example in DEVELOPMENT.md from bundler exec jekyll serve to bundle exec jekyll serve. That is one file and one line.
Checked against.
Date: Tue Oct 13 14:08:03 UTC 2026 (from date).
Repo grep for bundler exec|bundle exec in the checkout at cc22d0e (not an external fact).
unverified: "the Bundler CLI command is bundle exec" (PR body). I fetched no URL for it. I also did not check whether the bundler executable accepts exec as an alias.
Needs your decision.
pixi.toml:13-19 has seven tasks that still use bundler exec (serve, serve-dev, serve-fast, serve-minimal, build-dev, build-minimal, profile). The request names only DEVELOPMENT.md, so the PR leaves them as they are. Recommendation: if bundler exec is wrong, fix pixi.toml in a separate PR. If it works, the PR's rationale ("the Bundler CLI command is bundle exec") is about consistency only, and the PR text should say so. Either way I would not widen this PR.
Findings.
No blocking findings. The diff is one line, DEVELOPMENT.md:108, and bundle exec now matches every other doc and workflow command (README.md:191, TRANSLATION.md:62, tests/README.md:52, .github/workflows/*.yml). The sibling difference in pixi.toml is decision 1.
The PR body says "do not merge until the review comment has appeared". The PR is non-draft, so mergeability depends on you.
Tests. The change is docs-only, so no test is expected. There is no invariant test that runs over every instance, for example a check that no doc or config uses bundler exec. Existing suite output, literal (no pyproject.toml or uv.lock, so the npm path ran):
Test Suites: 15 passed, 15 total
Tests: 423 passed, 423 total
npm ci completed. npm test is jest. Ruff was not run because it is part of the uv path, which does not apply here.
Bounds. I read the full diff, the PR body and coding/skills/review/SKILL.md. I grepped the whole repo for bundler exec and bundle exec. I skimmed nothing else. I did not run pytest, ruff or the Playwright e2e tests, and the Jekyll build was not run (docs-only change). .github/claude-review.md does not exist. git rev-parse HEAD equals the payload Head SHA cc22d0e3bef9b05abb0a85238114753775a9405a.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DEVELOPMENT.md told readers to run
bundler exec jekyll servefor a custom config; the Bundler CLI command isbundle exec, matching the other examples in the file.Docs-only change. Opened to exercise the webhook-fired review routine; do not merge until the review comment has appeared.