Skip to content

Add .envrc to gitignore with example file #908

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

Closed
wants to merge 1 commit into from

Conversation

felixweinberger
Copy link
Contributor

@felixweinberger felixweinberger commented Jun 9, 2025

Summary

  • Added .envrc to .gitignore
  • Created .envrc.example as a template

Motivation

Providing an example file to help setup .venv automatically on cd with e.g. direnv.

Changes

  1. Updated .gitignore to exclude .envrc files
  2. Added .envrc.example with placeholder environment variables

Test plan

N/A

@felixweinberger felixweinberger marked this pull request as ready for review June 9, 2025 18:37
Added .envrc to .gitignore to prevent accidental commits of environment-specific
configurations. Included .envrc.example as a template for developers to copy and
customize with their local environment variables.

This ensures sensitive configuration remains local while providing clear guidance
on required environment setup.
@felixweinberger felixweinberger force-pushed the fweinberger/envrc-example branch from 4f2b0b5 to 0442470 Compare June 9, 2025 18:45
@Kludex
Copy link
Member

Kludex commented Jun 11, 2025

This PR seems to come from an Anthropic member, so I'll leave this to you folks.

I personally don't accept this kind of PR on the repositories I maintain because I believe the .gitignore should only contain what the repository itself builds/uses/generates that should be ignored. Any user specific files should be in their global .gitignore.

@Kludex Kludex deleted the fweinberger/envrc-example branch June 11, 2025 20:09
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.

2 participants