Skip to content

chore: adopt dryvist org-default gitignore#9

Merged
JacobPEvans-personal merged 1 commit into
mainfrom
chore/adopt-org-default-gitignore
Jun 18, 2026
Merged

chore: adopt dryvist org-default gitignore#9
JacobPEvans-personal merged 1 commit into
mainfrom
chore/adopt-org-default-gitignore

Conversation

@JacobPEvans-personal

@JacobPEvans-personal JacobPEvans-personal commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

Standardizes .gitignore coverage as part of an org-wide hygiene initiative.
Ensures common credential, secret-material, infrastructure-state, and tool-local
artifact paths are ignored consistently across dryvist repos, so files that
shouldn't be in version control aren't tracked by accident.

Adopts the relevant subset of the org-default baseline (dryvist/.github
configs/gitignore). Append-only and de-duplicated — no existing entries are
modified or removed. Verified that no currently-tracked file becomes ignored.

Intentional carve-outs preserved (committed by convention): .envrc,
*.sops.*, and .terraform.lock.hcl.

🤖 Generated with Claude Code

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

Copy link
Copy Markdown

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 exclude environment files, private keys, Ansible Vault files, generic secrets, cloud provider credentials, Doppler caches, Terraform states, and AI assistant local states. Feedback suggests refining several patterns: allowing encrypted Ansible Vault files (vault.yml/vault.yaml) to be committed, adding negations to prevent public keys (.pub) from being ignored, narrowing down the overly broad *credentials* pattern to avoid matching source code or documentation, and ensuring setup documentation like SECRETS_SETUP.md is not ignored.

Comment thread .gitignore
Comment thread .gitignore
Comment thread .gitignore
Comment thread .gitignore
@JacobPEvans-personal JacobPEvans-personal changed the title chore: adopt dryvist org-default gitignore (secrets + AI local state) chore: adopt dryvist org-default gitignore Jun 18, 2026
@JacobPEvans-personal
JacobPEvans-personal merged commit 8baee03 into main Jun 18, 2026
8 checks passed
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