-
-
Notifications
You must be signed in to change notification settings - Fork 2
Update installation instructions to use dev-road-runner
in docs, remove extra.branch-alias
from composer.json
.
#142
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
…ADME.md` to `Installation` section. (#141)
Warning Rate limit exceeded@terabytesoftw has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 1 minutes and 25 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (2)
WalkthroughAdds a new changelog entry for 0.1.1 (under development), repositions and updates README badges and installation commands to use dev-road-runner, updates docs installation methods to target dev-road-runner, and removes the composer extra.branch-alias (dev-main → 0.2.x-dev). Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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 (
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
…ion of `README.md`.
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: 2
🧹 Nitpick comments (1)
CHANGELOG.md (1)
3-6
: Expand entry to reflect all user-facing doc changes in this PR.The changelog mentions badge relocation but omits other doc-affecting changes (install commands now target dev-road-runner) and the removal of composer branch-alias. Consider listing them for clarity.
## 0.1.1 Under development - Bug #140: Move badges for latest stable version and total downloads in `README.md` to `Installation` section (@terabytesoftw) + - Docs: Update installation instructions to use `dev-road-runner` across README and docs (@terabytesoftw) + - Build: Remove `extra.branch-alias` (dev-main → 0.2.x-dev) from `composer.json` (@terabytesoftw)
📜 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 (4)
CHANGELOG.md
(1 hunks)README.md
(1 hunks)composer.json
(0 hunks)docs/installation.md
(2 hunks)
💤 Files with no reviewable changes (1)
- composer.json
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-08-25T21:03:02.960Z
Learnt from: terabytesoftw
PR: yii2-extensions/app-basic#128
File: docs/installation.md:7-7
Timestamp: 2025-08-25T21:03:02.960Z
Learning: In yii2-extensions/app-basic project, npm is listed as a system requirement in docs/installation.md because the composer.json file contains scripts that depend on npm commands for frontend asset management and build processes.
Applied to files:
README.md
🔇 Additional comments (1)
README.md (1)
72-75
: Remove explicit--stability=dev
: specifying the dev branch viacomposer create-project --prefer-dist yii2-extensions/app-basic:dev-road-runner app-basicalready installs the unstable package without requiring
--stability=dev
(drupal.org, github.com)Likely an incorrect or invalid review comment.
…n section of `README.md`.
@coderabbitai review |
✅ Actions performedReview triggered.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
✅ Actions performedReview triggered.
|
dev-road-runner
in docs, remove extra.branch-alias
from composer.json
.
Summary by CodeRabbit
Documentation
Chores