fix: shrink hero diagram + reconcile JDK/CI/ruleset facts#77
Merged
Conversation
Hero diagram: replace the Mermaid C4Container (rendered 747×1289 — a ~1289px-tall tower from C4's single-column layout + large fixed boxes) with a compact `flowchart LR` (988×373, ~3.5× shorter). Same content, clean landscape. Doc reconciliation (README + CLAUDE.md vs code / CI / ruleset): - Build Profiles "JDK" column showed 11-tem/17-tem/21-tem from the pom `jdk.version` property — VESTIGIAL (set 4×, referenced nowhere; the compiler uses maven.compiler.release) and contradicting the CI matrix. Removed the dead property; table now shows Java release (11/17/21 bytecode target) + CI-tested JDKs (11,18,25 / 18,25 / 25) matching ci.yml. - Prerequisites JDK row: 11/17/21 → "21 (Temurin), bytecode targets 11/17/21". - CI/CD job table omitted `static-check` (ci.yml has changes/static-check/ build/ci-pass) — added it. - ci-pass is the single required status check on master (repository ruleset, enforced by the owner); docs now state this accurately. Paired with Renovate platformAutomerge:false to also close the auto-merge registration race. - Renovate line → "automerges PRs once CI is green". Co-Authored-By: Claude Opus 4.8 (1M context) <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.
Addresses the reconciliation gaps surfaced in review (the diagram size + the JDK/CI doc drift), plus updates docs for the newly-added
ci-passruleset.Hero diagram
The Mermaid
C4Containerrendered 747×1289 (a ~1289px-tall tower — C4's single-column layout + large fixed boxes). Replaced with a compactflowchart LR(988×373, ~3.5× shorter). Same content, verified bymermaid-lint.Doc reconciliation (README + CLAUDE.md vs code / CI / ruleset)
jdk.version)11/17/21+ CI-tested JDKs11,18,25 / 18,25 / 25(matchesci.yml)static-check(ci.yml has 4 jobs)ci-passis required via repository ruleset; docs say so accuratelyrenovate.jsonplatformAutomerge: falsekept deliberately — required-check +platformAutomerge:falsetogether close both the gate and the auto-merge registration race.Verified
make cigreen (mermaid-lint on the new flowchart + build + test);make verify-allgreen on all 3 profiles after thejdk.versionremoval;ci-passruleset confirmed viagh api .../rules/branches/master.🤖 Generated with Claude Code