Skip to content

chore: add whitelist-style .gitignore#21

Open
LearningCircuit wants to merge 1 commit intomainfrom
chore/add-gitignore
Open

chore: add whitelist-style .gitignore#21
LearningCircuit wants to merge 1 commit intomainfrom
chore/add-gitignore

Conversation

@LearningCircuit
Copy link
Copy Markdown
Owner

Summary

  • Adds a .gitignore using a whitelist approach — deny everything by default, then explicitly allow only the file types and paths used in this repo
  • Allowed extensions: .sh, .yml, .yaml, .md, .txt
  • Allowed root files: LICENSE, README.md, .gitignore
  • Allowed directories: .github/ (.yml, .yaml, .md)
  • Explicitly blocks .venv/, IDE/OS artifacts, logs, temp files, and secrets

Test plan

  • All currently tracked files (ai-reviewer.sh, LICENSE, README.md, .github/workflows/ai-code-reviewer.yml) remain tracked
  • .venv/ does not appear in untracked files
  • .claude/settings.local.json is ignored (.json not in allow list)
  • test.txt appears as untracked (allowed by !*.txt)
  • test.py is ignored (.py not in allow list)
  • new_script.sh appears as untracked (allowed by !*.sh)

Use a deny-by-default approach that only allows specific file types
(.sh, .yml, .yaml, .md, .txt) and paths (LICENSE, .github/). Blocks
.venv/, IDE/OS artifacts, logs, temp files, and secrets.
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.

1 participant