docs: prepare the v0.2.3 release - #34
Merged
Merged
Conversation
Folds the demo recording and the config-path correction into the changelog, closes the [Unreleased] section as 0.2.3, and labels the README's status example as illustrative. That label matters more than it looks. The block sits directly beneath a GIF of real output, and its numbers are invented — 47 domains, three plausible expiry dates. Two adjacent things that look alike, one real and one not, with nothing saying which is which. Recording a real `namecom status` was the obvious alternative and is not usable: the sandbox account reports 6516 domains, a test name of mashed keyboard, and an expiry 741 days in the past. So the honest fix is the label, not a substitution. CLAUDE.md's config-path claim is corrected to match what README and the Path() doc comment now say. It is gitignored, so this commit does not carry it, but it was the last copy still asserting ~/.config on every platform — which is what sent an earlier search for credentials to a directory macOS never uses. The release itself is worth cutting rather than deferring: 0.2.2 users are still running the unvalidated `namecom open`, and the argument validation is the one change in this batch they cannot get any other way.
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes
[Unreleased]as 0.2.3, folding in the demo recording and the config-path correction that landed after the section was first written.Why cut this now
0.2.2 users are still running the unvalidated
namecom open. The argument validation is the one change in this batch they can't get any other way, and it's a behavior change (namecom open <not-a-domain>now errors) that deserves its own release note rather than riding along silently in a later one.15 commits since v0.2.2.
The README status block is now labelled
It sits directly beneath a GIF of real output, and its numbers are invented — 47 domains, three plausible expiry dates. Two adjacent things that look alike, one real and one not, with nothing saying which is which.
Recording a real
namecom statuswas the obvious alternative and isn't usable — the sandbox account reports:So the honest fix is a label, not a substitution.
Also (not in this diff)
CLAUDE.md's config-path claim is corrected locally to match what the README and thePath()doc comment now say. It's gitignored so it can't ride along here, but it was the last copy still asserting~/.configon every platform — the claim that sent an earlier credential search to a directory macOS never uses.Test plan
go build ./...,golangci-lint run,go test -race -count=1 ./...all clean[0.2.3]compare link added,[Unreleased]retargetednamecom statusoutput checked before deciding not to use itAfter merge
Tagging
v0.2.3triggersrelease.yml, which publishes binaries and pushes to the Homebrew tap. I won't push that tag without you saying so.