Skip to content

feat: gitignore-codegraph - #852

Merged
yxtay merged 1 commit into
mainfrom
feat/gitignore-codegraph
Jul 2, 2026
Merged

feat: gitignore-codegraph#852
yxtay merged 1 commit into
mainfrom
feat/gitignore-codegraph

Conversation

@yxtay

@yxtay yxtay commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Updated ignored files so local Claude settings at the repository root are excluded, and added the .codegraph/ directory to ignored items.

@yxtay
yxtay merged commit fb47098 into main Jul 2, 2026
14 of 16 checks passed
@yxtay
yxtay deleted the feat/gitignore-codegraph branch July 2, 2026 06:46
@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ed09354-2241-4a50-98af-96790e57dd7b

📥 Commits

Reviewing files that changed from the base of the PR and between 922eb7b and 466a1a6.

📒 Files selected for processing (1)
  • chezmoi/private_dot_config/git/ignore.tmpl

📝 Walkthrough

Walkthrough

Modifies the git ignore template to change the .claude/settings.local.json ignore pattern from a recursive glob to a root-level-only match, and adds a new ignore entry for the .codegraph/ directory.

Changes

Gitignore Template Update

Layer / File(s) Summary
Ignore pattern update
chezmoi/private_dot_config/git/ignore.tmpl
Changes **/.claude/settings.local.json to root-level .claude/settings.local.json and adds .codegraph/ as a new ignored directory.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • yxtay/dotfiles#832: Prior modification to the same .claude/settings.local.json ignore pattern in the same file.

Poem

A rabbit hops through config trees so neat,
Trimming globs to make the ignores petite,
.codegraph/ now hidden from sight,
settings.local.json scoped just right,
Thump thump — another tidy commit complete! 🐇

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/gitignore-codegraph

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.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the git ignore template by adding .codegraph/ and changing the ignore pattern for Claude local settings from **/.claude/settings.local.json to .claude/settings.local.json. The reviewer advises keeping the original wildcard pattern to ensure settings are ignored in nested directories and to prevent Claude Code from automatically re-adding the pattern and creating duplicate entries.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

{{- end }}

**/.claude/settings.local.json
.claude/settings.local.json

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.

medium

Changing **/.claude/settings.local.json to .claude/settings.local.json is not recommended for two reasons:

  1. Nested Projects / Monorepos: The **/.claude/settings.local.json pattern ensures that local settings are ignored at any directory depth, whereas .claude/settings.local.json only matches at the repository root.
  2. Automatic Management: Claude Code automatically checks for the exact pattern **/.claude/settings.local.json in the global gitignore file (~/.config/git/ignore). If it is modified or removed, Claude Code may automatically append the original pattern again, leading to duplicate entries.

It is best to keep the original pattern.

**/.claude/settings.local.json

@coderabbitai coderabbitai Bot mentioned this pull request Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant