Skip to content

chore: gitignore .envrc and .worktrees#269

Open
sanjeed5 wants to merge 2 commits intofederated-sdk-release-candidatefrom
chore/gitignore-envrc
Open

chore: gitignore .envrc and .worktrees#269
sanjeed5 wants to merge 2 commits intofederated-sdk-release-candidatefrom
chore/gitignore-envrc

Conversation

@sanjeed5
Copy link
Contributor

Summary

  • Stop tracking .envrc (local direnv config with dev-specific env profile paths) and add it to .gitignore
  • Add .worktrees to .gitignore

Each developer has their own direnv setup for API keys and env profiles — this shouldn't be in version control.

.envrc contains local direnv config (env profile paths, API keys)
that varies per developer and shouldn't be tracked. .worktrees is
a git worktree working directory.
@sanjeed5 sanjeed5 requested a review from a team February 25, 2026 13:07
@snyk-io
Copy link

snyk-io bot commented Feb 25, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@claude
Copy link
Contributor

claude bot commented Feb 25, 2026

Claude finished @sanjeed5's task in 38s —— View job


Review: PR #269chore: gitignore .envrc and .worktrees

  • Gather context and review changed files
  • Check for security concerns (leaked secrets in deleted .envrc)
  • Post review feedback

Verdict: Looks good

This is a clean housekeeping change. The deleted .envrc contained only direnv commands (use flake, dotenv_if_exists) — no secrets or credentials. The .gitignore already covers .direnv/ (line 147) and .env (line 113), so adding .envrc and .worktrees is the right complement.

No issues found — no impact on SDK code, public APIs, or backwards compatibility.

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