Skip to content

fix: default seed direnv to warn - #26

Merged
sourishkrout merged 1 commit into
mainfrom
trude/direnv-default-warn
Aug 7, 2026
Merged

sourishkrout merged 1 commit into
mainfrom
trude/direnv-default-warn

Conversation

@sourishkrout

Copy link
Copy Markdown
Contributor

Summary

  • default zero-value seed direnv policy to enabled_warn
  • keep explicit DirenvDisabled as the opt-out
  • cover seed.Options{} and embedded LocalStoreOptions{} behavior

Verification

  • go test ./internal/seed ./cmd -run 'Test(NewStore.*Direnv|DirenvPolicy|LocalStoreClient.*Direnv)' -count=1 -v
  • make check
  • runme run test
  • runme run lint

@sourishkrout
sourishkrout merged commit 13e696e into main Aug 7, 2026
2 checks passed
@sourishkrout
sourishkrout deleted the trude/direnv-default-warn branch August 7, 2026 19:55
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