chore(action): Polish review sticky comment with alerts and emoji#110
Merged
Conversation
Switch the sticky-comment envelope from a plain blockquote to GitHub Alert syntax (`[!CAUTION]` / `[!WARNING]` / `[!TIP]`) so blocking, non-blocking, and clean states render with their own coloured headers and icons. Each state now carries a leading emoji and a short call-to-action so the comment is easier to scan at a glance. Apply the same treatment to action/examples/relune-review.yaml so the sample workflow stays in sync with the dogfood path.
Code Metrics Report
Details | | main (24e0e79) | #110 (806722a) | +/- |
|---------------------|----------------|----------------|------|
| Coverage | 94.6% | 94.6% | 0.0% |
| Files | 81 | 81 | 0 |
| Lines | 36827 | 36827 | 0 |
| Covered | 34864 | 34864 | 0 |
- | Test Execution Time | 1m31s | 1m32s | +1s |Reported by octocov |
Schema reviewTip ✅ No risk findings — schema changes look safe to merge. |
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
[!CAUTION]/[!WARNING]/[!TIP]) so the three lifecycle states (blocking, non-blocking findings, clean) render with their own coloured headers and built-in icons.[!TIP]block so the comment never feels too sparse, with a leading emoji and short call-to-action on each state.action/examples/relune-review.yamlso the sample workflow stays in sync with the dogfood path under.github/workflows/dogfood-review.yaml.Changes
> ⚠️ **Blocking findings detected.**to a[!CAUTION]alert with a 🛑 emoji and "Resolve them before merging." follow-up.elifbranch that wraps non-blocking findings in a[!WARNING]alert with a 🔍 emoji prompting the reviewer to read the report below.✅ No risk findings.line with a[!TIP]alert so resolved PRs still get a visually consistent sticky comment instead of a one-line message.