Allow BC check to install dev dependencies#128
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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
This PR extends the reusable backward-compatibility (BC) workflow by adding an optional input to allow installing development dependencies when running roave/backward-compatibility-check, enabling accurate comparisons for packages whose public API references symbols from dev dependencies.
Changes:
- Added a new
install-development-dependenciesboolean input to the reusable workflow. - Updated the BC check command to conditionally pass
--install-development-dependenciesbased on the new input.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Adds an optional
install-development-dependenciesinput to the reusable BC workflow.\n\nWhen enabled, the workflow passes--install-development-dependenciestoroave/backward-compatibility-check. This lets consumers compare packages whose released API references symbols from development dependencies, while keeping the current default behavior unchanged.\n\nNeeded by yiisoft/validator#797, where Roave otherwise reports skipped debug collector symbols from the compared release.