Repository navigation
Pin Chek Reader to its 0.1.0 release instead of main - #4
Merged
Merged
Conversation
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XfwUCp8GqHWUdTK7EhZCT5
Any reader push could break the app's next build, and an App Store build wasn't reproducible. The project now takes 0.1.0 up to the next minor, and Package.resolved records that tag's commit. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XfwUCp8GqHWUdTK7EhZCT5
2 tasks done
Localizable.xcstrings was empty, so nothing in the app could be translated and there was no record of its strings. It now lists every key the code uses, with translator comments on the ones that are ambiguous on their own, and InfoPlist.xcstrings covers the local network usage description. Xcode keeps both up to date from here. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XfwUCp8GqHWUdTK7EhZCT5
Contributor
Author
|
The iOS app check fails at package resolution: no version of Generated by Claude Code |
…endabot' into claude/project-thread-eheuhr-strings
…ct-thread-eheuhr-reader-pin
devaryakjha
changed the base branch from
main
to
claude/project-thread-eheuhr-strings
October 21, 2026 04:51
devaryakjha
changed the base branch from
claude/project-thread-eheuhr-strings
to
main
October 21, 2026 04:52
2 tasks done
devaryakjha
marked this pull request as ready for review
October 21, 2026 05:00
The 0.1.0 tag landed on 0abd128, after the release workflow merged, not on the commit Package.resolved guessed before the tag existed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XfwUCp8GqHWUdTK7EhZCT5
This was referenced Oct 21, 2026
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.
Requested by Arya · project thread
Draft until chek-reader has a
0.1.0tag. My session can't push tags, so the tag is Arya's to create onmainat62271bb(the merge of chek-reader#26):Until then package resolution fails and CI is red, which is expected.
Before
The project took Chek Reader from
branch: main. Any reader push could break the app's next build, and an App Store build wasn't reproducible: two builds a day apart could ship different reader code.After
The project takes
0.1.0up to the next minor, andPackage.resolvedrecords that tag's commit. Taking a new reader release is a deliberate bump. The README says how, and how to work against unreleased reader changes with a local package override.How
The package requirement in
project.pbxprojand the pin inPackage.resolved, by hand, since Xcode isn't available in this session. CI's iOS build is the check.Checks
Generated by Claude Code