Conversation
…dule - Add module-level and item-level doc comments across all src/ui/ files - Fix print_warn writing to stdout instead of stderr - Make ApplySummary::print respect --quiet via UiContext param - Add bounds check on select default in --yes mode - Add Spinner::warn() method and Drop impl for early-return safety - Fix ASCII header alignment and add version tag - Extract INDENT constant from hardcoded spaces - Use explicit match arm + unreachable!() in conflict_resolution
Run cargo check, clippy, and fmt in parallel on PRs to master using stable Rust toolchain with dependency caching.
Drop redundant cargo check (clippy subsumes it), merge 3 separate runners into one job, add cargo test, and order steps so fmt fails fast before compilation. Also adds push trigger on master and least-privilege permissions.
Add .claude/ to .gitignore and untrack agent configs. These are local development files that shouldn't be shared.
Step 3 (UI Module) was already implemented but shown as incomplete in README. Phase reference was pointing to Phase 3 (Polish & Distribution) instead of Phase 1 (Interactive MVP). CI workflow was already added but not reflected in ROADMAP checklist.
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.
Changes