Skip to content

chore: add .env.enterprise and reports/ to .gitignore#800

Open
ltianyi992 wants to merge 1 commit into
TauricResearch:mainfrom
ltianyi992:fix/gitignore-env-enterprise-reports
Open

chore: add .env.enterprise and reports/ to .gitignore#800
ltianyi992 wants to merge 1 commit into
TauricResearch:mainfrom
ltianyi992:fix/gitignore-env-enterprise-reports

Conversation

@ltianyi992
Copy link
Copy Markdown

Closes #787

Summary

  • Adds .env.enterprise to prevent accidental commit of enterprise environment secrets
  • Adds reports/ to prevent committing generated report output files

Test plan

  • Verify .env.enterprise is ignored by git
  • Verify reports/ directory is ignored by git

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the .gitignore file to include .env.enterprise and a new reports/ directory for generated reports. The review feedback suggests using a wildcard pattern .env.* to more robustly exclude environment-specific configuration files and prevent the accidental exposure of sensitive credentials.

Comment thread .gitignore

# Environments
.env
.env.enterprise
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

security-medium medium

Using a wildcard pattern like .env.* is more robust than listing specific environment files. This ensures that any environment-specific configuration files (e.g., .env.local, .env.production) are automatically ignored, reducing the risk of accidentally committing sensitive credentials.

.env.*

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.

Gitignore .env.enterprise and reports folder

1 participant