Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe README development setup instructions were updated to use the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the development setup instructions so contributors install the dependencies needed to run the full test suite (including the CLI-related optional dependencies).
Changes:
- Update the README’s Poetry install command to install extras required by CLI tests.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```bash | ||
| npm install | ||
| poetry install | ||
| poetry install --all-extras |
There was a problem hiding this comment.
README now recommends poetry install --all-extras, but the repo’s GitHub workflows install only the cli extra (poetry install --extras cli ...). Consider switching to --extras cli here to keep docs aligned with CI and avoid installing unrelated optional dependencies if more extras are added later.
| poetry install --all-extras | |
| poetry install --extras cli |
Proposed Changes
We need to install
cliextras to run all tests.Without CLI extras, tests failed with the following error:
Related Issues
Summary by CodeRabbit