Skip to content

chore: ignore .env so a local token cannot be staged - #39

Open
sergebulaev wants to merge 1 commit into
mainfrom
chore/gitignore-env
Open

sergebulaev wants to merge 1 commit into
mainfrom
chore/gitignore-env

Conversation

@sergebulaev

Copy link
Copy Markdown
Contributor

.env is not in .gitignore here, so git add -A stages it.

There is no application code in this repo that reads a .env, so one in a working copy is somebody's own tooling credentials. GitHub push protection has already had to stop a GitHub PAT in one from reaching a branch during work on #25 today — it caught it, nothing reached the remote, and that is the good outcome. This makes the earlier step impossible rather than relying on the later one.

.env.example stays allowed, in case one is ever wanted.

A .env in a working copy of this repo holds somebody's own tooling credentials -
there is no application code here that would read one. It was not ignored, so
`git add -A` stages it, and GitHub push protection has already had to stop a
GitHub PAT in one from reaching a branch.

Push protection caught it, which is the good outcome. This makes the earlier
step impossible instead of relying on it.
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