Skip to content

BAI-339 gitignore Claude Code local settings and worktree checkouts - #60

Merged
imranq2 merged 2 commits into
mainfrom
IQ-BAI-339
Jul 24, 2026
Merged

BAI-339 gitignore Claude Code local settings and worktree checkouts#60
imranq2 merged 2 commits into
mainfrom
IQ-BAI-339

Conversation

@imranq2

@imranq2 imranq2 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • .claude/settings.local.json and .claude/worktrees/ were showing up as untracked in git status — the former is a personal local override, the latter holds actual git worktree checkouts (each with its own .git pointer file) that should never be tracked in the parent repo.
  • Adds both to .gitignore, matching the same pattern applied to baileyai.

Test plan

  • git status no longer lists .claude/ as untracked.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM — straightforward .gitignore addition.

Extended reasoning...

Overview

The PR adds two entries to .gitignore: .claude/settings.local.json (a personal local override file) and .claude/worktrees/ (git worktree checkouts). This is the only change in the PR — no application code, configuration, or infrastructure is touched.

Security risks

None. Ignoring local Claude Code settings and worktree directories reduces the risk of accidentally committing personal/local state; it does not affect auth, PHI/PII handling, or any data path.

Level of scrutiny

Minimal scrutiny is appropriate here — this is a purely additive, mechanical change to a dotfile with no behavioral, architectural, or contract impact. It mirrors an existing pattern already applied in another repo (baileyai), so there is no novel decision to review.

Other factors

No tests are needed for a .gitignore change, and the PR's own manual test plan (confirming git status no longer lists .claude/ as untracked) is sufficient verification. No outstanding review comments exist on this PR.

@alvinhenrick
alvinhenrick self-requested a review July 24, 2026 21:59
@imranq2
imranq2 merged commit 3118680 into main Jul 24, 2026
3 checks passed
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