Repository navigation
docs: architecture/sequence/module diagrams, real CLI screenshots, badges, CI - #25
Merged
Merged
Conversation
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.
Visual pass across the repo, per Joy's request to add visuals wherever a reviewer would actually look.
Verified: all 55 tests still pass; the new tests.yml and existing daily-fetch.yml both validated as syntactically correct YAML; the asset-generation script actually run end-to-end (confirmed real 5/1/4 fetch/new/filtered counts and a genuine DB-vs-Excel discrepancy in the reconcile screenshot, not staged data).
One thing I could not verify myself: Mermaid diagram rendering. I applied defensive quoting throughout (any label with parens, arrows, or special characters) since a few of GitHub's Mermaid parsing edge cases aren't something I can render-test from here — please check the 4 diagrams actually render correctly once this PR is viewable on GitHub before merging.