-
-
Notifications
You must be signed in to change notification settings - Fork 0
fix(docs): Reorganize badge section and update headings in README.md.
#168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughREADME.md header badges were reorganized: the old centered Development Status badge block was removed; a new "Package information" heading with Development Status and Total Downloads badges was added; the "Quality code" section header was reintroduced, retaining existing Codecov, PHPStan, and StyleCI badges. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
README.md (1)
16-18: Broken/invalid Yii version badge and link (“22.0.x”).
yiisoft/yii2does not have a “22.0.x” series; the link path/tree/22.0is invalid and the badge is misleading. Either remove this badge or correct it. Given you already have a “2.0.x” badge on Lines 13–15, removal is likely best.- <a href="https://github.com/yiisoft/yii2/tree/22.0" target="_blank"> - <img src="https://img.shields.io/badge/22.0.x-0073AA.svg?style=for-the-badge&logo=yii&logoColor=white" alt="Yii 22.0.x"> - </a>
🧹 Nitpick comments (1)
README.md (1)
52-56: Good reorganization; consider adding Version and License badges.Nice consolidation under “Package information.” To help users quickly see the current release and license, add Packagist Version and License badges.
## Package information -[](https://packagist.org/packages/yii2-extensions/psr-bridge) +[](https://packagist.org/packages/yii2-extensions/psr-bridge) +[](https://packagist.org/packages/yii2-extensions/psr-bridge) [](https://packagist.org/packages/yii2-extensions/psr-bridge) + [](LICENSE.md)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📚 Learning: 2025-08-26T15:37:52.160Z
Learnt from: terabytesoftw
PR: yii2-extensions/psr-bridge#155
File: tests/http/stateless/ApplicationMemoryTest.php:198-211
Timestamp: 2025-08-26T15:37:52.160Z
Learning: In yii2-extensions/psr-bridge, tests can use the TestSupport trait from php-forge/support package which provides an invokeMethod() helper for calling protected/private methods during testing, eliminating the need for custom reflection code.
Applied to files:
README.md
📚 Learning: 2025-08-25T16:08:54.379Z
Learnt from: terabytesoftw
PR: yii2-extensions/psr-bridge#149
File: tests/http/stateless/ApplicationCoreTest.php:24-27
Timestamp: 2025-08-25T16:08:54.379Z
Learning: In the yii2-extensions/psr-bridge project, avoid importing functions that are not actually used in the code, even if they might seem logically related to the functionality being tested.
Applied to files:
README.md
📚 Learning: 2025-08-10T13:15:30.396Z
Learnt from: terabytesoftw
PR: yii2-extensions/psr-bridge#78
File: tests/adapter/ServerParamsPsr7Test.php:8-8
Timestamp: 2025-08-10T13:15:30.396Z
Learning: In the yii2-extensions/psr-bridge project, classes referenced in PHPDoc annotations (such as throws declarations) must be imported even if not used in the actual code, to avoid ghost class references and maintain proper documentation.
Applied to files:
README.md
🔇 Additional comments (1)
README.md (1)
57-58: Section split improves clarity.Reintroducing “Quality code” as a distinct section reads better and groups badges logically.
Summary by CodeRabbit