release: 0.1.7 - #6
Merged
Merged
Conversation
Marketing version to 0.1.7 and the build number to 8. Sparkle compares the build number, so that one is not optional: leaving it at 7 ships a release no installed app would offer. The changelog entry covers the four changes merged today, written for the person reading the update prompt rather than the diff. This commit only prepares the release. Tagging is what ships it. Claude-Session: https://claude.ai/code/session_01UraqY8Ld2E3zzGVN636Lw7
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
1 participant
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.
Prepares 0.1.7. Merging this does not ship anything. Pushing the tag does.
What it changes
project.ymland the generatedCopy/Info.plist: marketing version to0.1.7, build number7to8. Sparkle compares the build number, so leaving it at 7 would publish a release no installed app offers. Verified by building: the app reports0.1.7 build 8.CHANGELOG.md: a[0.1.7]entry for the four PRs merged today, written for whoever reads the update prompt.Matches the shape of the last bump (27a8068), which also committed the generated Info.plist.
Ship it with
release.ymlthen runs the tests, archives, signs, notarizes, staples, builds the DMG, publishes the release, updates the appcast and the Homebrew tap. Installed apps check hourly, so it reaches people within the hour.Why this is a PR and not a tag
The 187 tests are all CopyCore. The app target has no test target, and three of today's four changes (the shortcuts, the sounds, the dismissal rework) live entirely in the app target. So CI passing says nothing about them.
The dismissal rework in particular changed when the paste keystroke fires, which is the one thing this app must not get wrong, and it has run on one machine for a couple of hours. Worth a day of real use before it reaches everyone within the hour.
One thing I noticed but did not change
release.ymlpasses--notes-file CHANGELOG.md, so every GitHub release gets the entire changelog as its notes, not just the new section. Pre-existing, and not something to fix inside a release commit.https://claude.ai/code/session_01UraqY8Ld2E3zzGVN636Lw7