Skip to content

fix(cli): preserve explicit empty notify messages - #19

Open
taigrr wants to merge 1 commit into
masterfrom
cd/round7-quality
Open

taigrr wants to merge 1 commit into
masterfrom
cd/round7-quality

Conversation

@taigrr

@taigrr taigrr commented Sep 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • distinguish omitted notify messages from explicitly provided empty messages
  • keep piper notify name "" from falling back to STDIN
  • add unit coverage for notify argument handling

Validation

  • go generate ./...
  • go build ./...
  • go vet ./...
  • staticcheck ./...
  • go test -race ./...
  • go mod tidy && git diff --exit-code -- go.mod go.sum
  • govulncheck ./...

Notes: latest Go checked: go1.27.1. Existing Dependabot PR #18 is open, mergeable, and green, so this PR avoids touching logrus/go.sum.

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