Skip to content

Fix .claude global gitignore - #405

Open
gsnider2195 wants to merge 4 commits into
developfrom
u/gas-claude-gitignore
Open

Fix .claude global gitignore#405
gsnider2195 wants to merge 4 commits into
developfrom
u/gas-claude-gitignore

Conversation

@gsnider2195

Copy link
Copy Markdown
Contributor

Closes N/A

What's Changed

  • Removed CLAUDE.md and .claude/ from the ignore list
  • Added .claude/settings.local.json and CLAUDE.local.md

Claude Code distinguishes between team-shared and personal configuration:

  • Shared (should be committed): CLAUDE.md (project instructions), .claude/settings.json (permissions, hooks, additional directories), and .claude/commands/, agents/, skills/, rules/. Checking these in gives every developer on a project the same instructions and permission baseline, which is a prerequisite for standardizing Claude Code behavior across our repos.
  • Personal (should stay ignored): .claude/settings.local.json (per-user overrides; also where "Yes, don't ask again" approvals accumulate) and CLAUDE.local.md (per-user instructions).

To Do

  • Update the documentation.
  • Add changelog.
  • Verify your changes by testing them in the dev-example repository before merging.


# Claude Code
CLAUDE.md
.claude/

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.

So there are items that will be in .claude/ that we do want to share?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah stuff like per-project skills, agents, system prompt modifications.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I realized I just restated what was in the summary. A good use case is a claude skill in the chatops app to scaffold a new chatops integration or in ssot for a new ssot integration. Those would be good to stay in the repo instead of our global claude repo.

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