Skip to content

Add .gitignore entries for sensitive files#193

Open
pete-builds wants to merge 3 commits intonode-strava:mainfrom
pete-builds:gitignore-env-and-secrets
Open

Add .gitignore entries for sensitive files#193
pete-builds wants to merge 3 commits intonode-strava:mainfrom
pete-builds:gitignore-env-and-secrets

Conversation

@pete-builds
Copy link
Copy Markdown

@pete-builds pete-builds commented Apr 12, 2026

The .gitignore is missing entries for .env files and private keys (*.pem, *.key). Added those to help prevent accidental commits of credentials.

Summary by CodeRabbit

  • Chores
    • Updated repository ignore rules to exclude environment-related files and common OS metadata from version control. This reduces noise in tracked changes and keeps the repository contents more focused and maintainable. No runtime behavior or public interfaces were modified; this is purely a repository maintenance change.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 86130104-3035-4f95-92eb-54bdf16c285c

📥 Commits

Reviewing files that changed from the base of the PR and between 2c9cbdb and 5fa8505.

📒 Files selected for processing (1)
  • .gitignore
✅ Files skipped from review due to trivial changes (1)
  • .gitignore

📝 Walkthrough

Walkthrough

The .gitignore was updated to add patterns excluding environment/secret-related files (.env, .env.* except !.env.example) and common OS metadata files (.DS_Store, Thumbs.db) from version control.

Changes

Cohort / File(s) Summary
Git ignore
.gitignore
Added ignore patterns for environment files (.env, .env.* with an explicit !.env.example exception) and OS-specific metadata (.DS_Store, Thumbs.db).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hide the secrets, soft and small,
.env tucked underneath my hall,
No .DS_Store crumbs along the way,
Clean commits hop in, bright as day,
A quiet code-burrow where I play.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change—adding .gitignore entries for sensitive files like .env and OS metadata files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
.gitignore (1)

40-42: Consider un-ignoring .env.example to preserve tracked templates.

Line 41 (.env.*) will also ignore .env.example. If this repo uses an example env template for onboarding, add an explicit negation rule.

Suggested tweak
 .env
 .env.*
+!.env.example
 *.pem
 *.key
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.gitignore around lines 40 - 42, The current ignore pattern ".env.*" will
also exclude the tracked template ".env.example"; update the .gitignore by
adding an explicit negation for the example file (e.g., add a pattern to
un-ignore ".env.example") placed after the ".env.*" entry so the template
remains tracked and other env variants stay ignored.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.gitignore:
- Around line 40-42: The current ignore pattern ".env.*" will also exclude the
tracked template ".env.example"; update the .gitignore by adding an explicit
negation for the example file (e.g., add a pattern to un-ignore ".env.example")
placed after the ".env.*" entry so the template remains tracked and other env
variants stay ignored.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 693ce2c2-f667-46c7-a3bb-05083f941fdd

📥 Commits

Reviewing files that changed from the base of the PR and between fff8fbc and 0fec414.

📒 Files selected for processing (1)
  • .gitignore

@pete-builds
Copy link
Copy Markdown
Author

Revised — removed the credential and certificate entries that aren't relevant to this project.

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