Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add *.env fils to .gitignore #67

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Conversation

lechnerc77
Copy link
Member

Purpose

  • Add *.env to the gitignore file to avoid committing files containing sensitive environment variables

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe:

How to Test

Create a *.env file in the directory structure of the project e.g. a devcontainer.env file

What to Check

Verify that the following are valid:

  • *.env file is excluded from the commit

Other Information

n/a

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

  • The PR is assigned to the Terraform project and a status is set (typically "in review").
  • The PR has the matching labels assigned to it.
  • The PR has a milestone assigned to it.
  • If the PR closes an issue, the issue is referenced.
  • Possible follow-up items are created and linked.

@lechnerc77 lechnerc77 self-assigned this Apr 4, 2024
@lechnerc77 lechnerc77 added the enhancement New feature or request label Apr 4, 2024
@Gourab1998 Gourab1998 merged commit d0e832a into main Apr 4, 2024
3 checks passed
@Gourab1998 Gourab1998 deleted the chore/adjust-gitignore branch April 4, 2024 10:28
Dray56 pushed a commit that referenced this pull request May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants