-
-
Notifications
You must be signed in to change notification settings - Fork 2
fix: Update license badge in README.md
for correct display and add missing header in LICENSE.md
.
#66
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
…missing header in `LICENSE.md`.
WalkthroughThis update modifies documentation files by adding a header to LICENSE.md, updating the license badge in README.md to use shields.io, and recording several bug fixes and documentation improvements in CHANGELOG.md. It also simplifies GitHub Actions workflows by removing custom composer command inputs and secrets. No changes were made to code or exported/public entities. Changes
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
💤 Files with no reviewable changes (3)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (18)
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. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #66 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 71 71
===========================================
Files 2 2
Lines 129 129
===========================================
Hits 129 129 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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
🧹 Nitpick comments (1)
README.md (1)
212-212
: Optional: make the badge alt-text more specific
alt="License"
is fine, but something likealt="BSD 3-Clause License"
would improve accessibility for screen-reader users and differentiate it from other badges.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
CHANGELOG.md
(1 hunks)LICENSE.md
(1 hunks)README.md
(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: terabytesoftw
PR: yii2-extensions/phpstan#40
File: src/ServiceMap.php:0-0
Timestamp: 2025-06-14T17:41:48.820Z
Learning: The yii2-extensions/phpstan repository contains a PHPStan extension specifically designed for static analysis of Yii framework applications.
README.md (4)
Learnt from: terabytesoftw
PR: yii2-extensions/phpstan#40
File: src/ServiceMap.php:0-0
Timestamp: 2025-06-14T17:41:48.820Z
Learning: The yii2-extensions/phpstan repository contains a PHPStan extension specifically designed for static analysis of Yii framework applications.
Learnt from: terabytesoftw
PR: yii2-extensions/phpstan#47
File: src/type/ActiveRecordGetAttributeDynamicMethodReturnTypeExtension.php:107-110
Timestamp: 2025-06-17T22:22:55.915Z
Learning: In PHPStan extensions for this repository, `instanceof` must not be used in PHPStan runtime mode; instead, use `::class` comparisons for reliable type checks (e.g., `$obj::class === SomeClass::class` instead of `$obj instanceof SomeClass`).
Learnt from: terabytesoftw
PR: yii2-extensions/phpstan#36
File: src/type/ActiveQueryDynamicMethodReturnTypeExtension.php:153-157
Timestamp: 2025-06-12T16:41:20.240Z
Learning: In PHPStan extensions for this repository, `instanceof` must not be used to compare `PHPStan\Type\*` objects (e.g., `ThisType`) because it is not compatible with PHPStan’s runtime mode; instead, use `$type::class === SomeType::class` for reliable checks.
Learnt from: terabytesoftw
PR: yii2-extensions/phpstan#60
File: composer.json:14-15
Timestamp: 2025-06-27T08:46:56.224Z
Learning: The yiisoft/yii2 package has a 22.0 development branch available on Packagist as 22.0.x-dev, making the version constraint ^22 valid in composer.json files.
🔇 Additional comments (2)
LICENSE.md (1)
1-2
: License header correctly addedThe added Markdown H-1 heading concisely labels the license and improves readability. No further action required.
CHANGELOG.md (1)
6-9
: Changelog entries look goodNew bug-fix items are clear, properly numbered, and follow the existing style.
Summary by CodeRabbit