Add status page badges - #430
Conversation
6555490 to
86288ac
Compare
86288ac to
844bfd6
Compare
There was a problem hiding this comment.
馃挕 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 844bfd6afd
鈩癸笍 About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 馃憤.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "require": { | ||
| "php": "^8.3", | ||
| "ext-simplexml": "*", | ||
| "cachethq/badger": "^5.0", |
There was a problem hiding this comment.
Obtain approval before adding Badger
This introduces cachethq/badger as a new runtime dependency, but the repository explicitly prohibits adding dependencies without prior approval, and the reviewed change provides no evidence of that approval. Obtain and document approval or use an existing approved dependency before merging.
AGENTS.md reference: AGENTS.md:L20-L24
Useful? React with 馃憤聽/ 馃憥.
Summary
Adds cacheable SVG badges for the overall status page and individual public components.
/badge.svgand/components/{component}/badge.svgbeneath the configured Cachet path.?style=.Validation
vendor/bin/pest tests/Feature/StatusPage/StatusPageTest.php tests/Unit/StatusBadgeColorTest.php --compact(33 passed)vendor/bin/pintfor changed filescomposer validate --no-check-publishThe full parallel test command was also attempted earlier but an unrelated subscriber test worker exceeded its fixed 128 MB memory limit.