Repository navigation
feat: add emojis to CLI output - #2852
Conversation
|
👋 Tofel, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
📊 API Diff Results
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The formatting changes are consistent, documented, and adequately tested.
Review effort: Balanced
Findings: None
What changed in this PR
Improves Grafana alert-check CLI readability with clearer spacing, labels, and status emojis.
Changes:
- Adds a
Legend:heading and separates the Grafana version. - Marks violation totals with ✅ or ❌.
- Updates tests, documentation, and release notes.
| File | Description |
|---|---|
grafana-alertcheck/cmd/table.go |
Refines CLI footer formatting and status markers. |
grafana-alertcheck/cmd/table_test.go |
Tests the revised output and emoji selection. |
grafana-alertcheck/docs/reference/cli.md |
Documents the updated human-readable output. |
grafana-alertcheck/.changeset/v0.1.5.md |
Records the user-facing CLI changes. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Make the output a bit cleaner + add emojis: