Skip to content

Conversation

@shaanmajid
Copy link
Contributor

@shaanmajid shaanmajid commented Dec 8, 2025

.claude/settings.local.json contains machine-specific paths and permissions. Cloning the repo fresh shows warnings for paths that don't exist.

This adds it to .gitignore and removes it from tracking.

Fixes #154

This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
@coderabbitai
Copy link

coderabbitai bot commented Dec 8, 2025

Walkthrough

A local Claude settings configuration file was removed from the repository, and the .gitignore was updated to prevent future commits of this file type, eliminating any locally-enforced permissions rules and directory configurations.

Changes

Cohort / File(s) Summary
Configuration cleanup
\.claude/settings.local.json
Deleted local settings file that contained permissions configuration (allow, deny, ask rules) and directory exclusions.
Git ignore update
\.gitignore
Added .claude/settings.local.json entry to prevent version control tracking of local settings files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A settings file bids its farewell,
Local permissions no longer to dwell,
In .gitignore it shall rest,
Clean repos are truly the best!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding .claude/settings.local.json to .gitignore while removing the file from version control.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@obra obra merged commit 4ac6783 into obra:main Dec 9, 2025
1 check passed
@obra
Copy link
Owner

obra commented Dec 9, 2025

thanks

mthalman pushed a commit to mthalman/superpowers that referenced this pull request Dec 19, 2025
This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
# Conflicts:
#	.claude/settings.local.json
pcvelz pushed a commit to pcvelz/superpowers that referenced this pull request Jan 30, 2026
This file contains machine-specific paths and permissions that cause
warnings when cloning the repo fresh.
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.

.claude/settings.local.json should be gitignored

2 participants