chore(ci): prepend pass/time/delta summary to test results comment#24430
Draft
Artur- wants to merge 4 commits into
Draft
chore(ci): prepend pass/time/delta summary to test results comment#24430Artur- wants to merge 4 commits into
Artur- wants to merge 4 commits into
Conversation
Test Results❌ 1 test failed Full report1 211 files - 209 1 211 suites - 209 1h 33m 45s ⏱️ + 13m 16s For more details on these failures, see this check. Results for commit c799658. ± Comparison against base commit 6bad3ec. ♻️ This comment has been updated with latest results. |
Lead with three concise lines (status, time vs reference, test count delta) and tuck EnricoMi's verbose table inside a <details> block so PR reviewers can see the headline at a glance.
EnricoMi's `json` step output only carries deltas when comparing against an earlier commit's check run, which doesn't exist on the first push to a new branch. The PR comment uses a separate base-commit comparison and renders the deltas straight into its text. Parse them from there.
Demonstration commit for the new EnricoMi summary header — triggers all three "things go wrong" lines (❌ test failed,⚠️ time over budget,⚠️ test removed). Revert before merging.
A 109.74% run rounds to "110%" on display but stayed green under the strict `>` check, which reads as a UI bug to anyone reading the threshold. Use `>=` so the rounded integer drives the warning.
Artur-
force-pushed
the
better-error-report
branch
from
57d7e64 to
c799658
Compare
|
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.
Lead with three concise lines (status, time vs reference, test count delta) and tuck EnricoMi's verbose table inside a
Details
block so PR reviewers can see the headline at a glance.