Conversation
Two bugs discovered during the 1.0.1 release caused the `update-homebrew`
job to fail; manual tap push was required to ship.
BUG-R1 — update-homebrew job could not find the template:
The job checks out only `randlee/homebrew-tap` at path `homebrew-tap`,
but the render step reads `release/homebrew/sc-compose.rb.j2` from the
sc-compose repo which is never checked out. Add an `actions/checkout@v4`
step with `path: sc-compose` and update the template path accordingly.
BUG-R2 — formula template install block broken:
Homebrew auto-cd's into the single top-level directory of the tarball
before running `def install`, so `Dir["sc-compose_*"].first` returns nil
and `"#{nil}/bin/sc-compose"` resolves to "/bin/sc-compose" (ENOENT).
Switch to flat paths (`bin/sc-compose`, `share/sc-compose/examples/*`)
matching the working manual tap formulas for 1.0.0 and 1.0.1.
With both fixes the next release should render and push the tap formula
automatically without manual intervention.
Co-authored-by: Rand Lee <randlee@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
* feat(H1): structured object input support (FR-12) (#45) * feat(html-report): add H1 object input support * docs: align H1 structured-input docs * Fix H1 QA follow-up cleanup --------- Co-authored-by: Rand Lee <randlee@users.noreply.github.com> * feat(H2): arrays of objects input support (FR-13) (#47) * feat: add H2 arrays-of-objects support * Fix HTML H2 QA findings --------- Co-authored-by: Rand Lee <randlee@users.noreply.github.com> * feat(H3): sprint-report-html bundled example (FR-14, FR-15) (#48) * Add H3 HTML sprint report example * Fix HTML H3 QA findings * Remove duplicate S8 plan block --------- Co-authored-by: Rand Lee <randlee@users.noreply.github.com> * feat(H4): multi-panel report and wrapper integration (#49) * Add H3 HTML sprint report example * Fix HTML H4 QA findings * Remove duplicated HTML example tests --------- Co-authored-by: Rand Lee <randlee@users.noreply.github.com> * Fix HTML phase-end findings and scope drift * Reduce include expansion PathBuf cloning (#51) Co-authored-by: Rand Lee <randlee@users.noreply.github.com> --------- Co-authored-by: Rand Lee <randlee@users.noreply.github.com>
Co-authored-by: Rand Lee <randlee@users.noreply.github.com>
Addresses REQ-QA-002: FR-12 through FR-15 coverage matrix was missing the Sprint H4 row. H4 introduced no new functional requirements — it extends FR-12–FR-15 with wrapper integration and doc finalization. Co-authored-by: Rand Lee <randlee@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Phase HTML-Report (FR-12–FR-15) shipped. Includes doc fixes for requirements.md section header and FR Coverage Matrix completeness. Co-authored-by: Rand Lee <randlee@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Summary
sprint-report-htmlbundled exampleinclude.rshot pathPRs included
Test plan
🤖 Generated with Claude Code