Skip to content

chore: ignore .claude and .codex directories#188

Merged
marcellodebernardi merged 2 commits intomainfrom
codex/ignore-claude-codex-dirs
Mar 5, 2026
Merged

chore: ignore .claude and .codex directories#188
marcellodebernardi merged 2 commits intomainfrom
codex/ignore-claude-codex-dirs

Conversation

@marcellodebernardi
Copy link
Contributor

This PR adds directory-level ignore rules for .claude/ and .codex/ in .gitignore. The aim is to prevent local agent/editor metadata folders from being accidentally committed while keeping the repository clean for all contributors.

Testing

  • git check-ignore -v .claude/test.txt .codex/test.txt

Copilot AI review requested due to automatic review settings March 5, 2026 15:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 5, 2026

Greptile Summary

This PR updates .gitignore to ignore the .claude/ and .codex/ directories at any depth in the repository, replacing the previous narrower **/.claude/settings.local.json rule. The change is minimal, correct, and achieves its stated goal of preventing local AI-agent/editor metadata from being accidentally committed.

  • .claude/ and .codex/ patterns correctly use the trailing-slash convention to match directories at any level of the tree.
  • No section comment/header was added for these new rules (unlike other entries such as # Experiments), but this is a minor style point and consistent with adjacent entries like config.yaml and Makefile.local.

Confidence Score: 5/5

  • Safe to merge — purely additive gitignore change with no risk of breaking anything.
  • The change touches only .gitignore, adds two well-formed directory-ignore patterns, and correctly removes the now-redundant narrower rule that was flagged in a previous review. There are no logic, syntax, or security concerns.
  • No files require special attention.

Important Files Changed

Filename Overview
.gitignore Replaces the narrower **/.claude/settings.local.json rule with broad .claude/ and .codex/ directory-level ignores. The patterns are syntactically correct and will match the named directories at any depth in the tree.

Last reviewed commit: 3a1f61e

@marcellodebernardi
Copy link
Contributor Author

@greptileai please review again with latest changes

@marcellodebernardi marcellodebernardi merged commit b2ebc97 into main Mar 5, 2026
13 checks passed
@marcellodebernardi marcellodebernardi deleted the codex/ignore-claude-codex-dirs branch March 5, 2026 15:51
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