Skip to content

chore: add ggshield cache to gitignore#14

Merged
rodaddy merged 1 commit intomainfrom
feat/apple-services
Mar 30, 2026
Merged

chore: add ggshield cache to gitignore#14
rodaddy merged 1 commit intomainfrom
feat/apple-services

Conversation

@rodaddy
Copy link
Copy Markdown
Owner

@rodaddy rodaddy commented Mar 30, 2026

Summary

  • Adds .cache_ggshield to .gitignore (auto-added by ggshield)

Related

Apple services (calendar, notes, reminders) were added to ~/.config/mcp2cli/services.json and skills generated -- those are config-level, not in this repo.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rodaddy rodaddy merged commit 68bf41b into main Mar 30, 2026
3 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 30, 2026

Claude finished @rodaddy's task —— View job


PR Review #14: chore: add ggshield cache to gitignore

  • Read relevant files
  • Analyze the change
  • Post review feedback

This is a minimal, clean change. Review against the stated criteria:

Bugs / Logic Errors: None. The entry correctly excludes .cache_ggshield from version control.

Security: Positive security change. ggshield is a secret-scanning tool; excluding its cache directory prevents accidental commit of scanner state files. No concerns.

Resource Leaks / Connections / Timeouts: Not applicable to a .gitignore change.

Error Handling: Not applicable.

TypeScript Strictness: Not applicable.

Style/Convention: The addition follows the existing structure — placed under the # caches section with an explanatory comment (# Added by ggshield). The comment accurately describes the source (auto-added by ggshield) which is helpful context for contributors.

No issues found. This is a routine housekeeping change that correctly prevents tool-generated cache files from being tracked. ✓

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