Skip to content

Remove BC dev dependency input#129

Merged
samdark merged 1 commit into
masterfrom
remove-bc-install-development-dependencies
May 31, 2026
Merged

Remove BC dev dependency input#129
samdark merged 1 commit into
masterfrom
remove-bc-install-development-dependencies

Conversation

@samdark
Copy link
Copy Markdown
Member

@samdark samdark commented May 31, 2026

Summary

  • remove the install-development-dependencies input from the reusable BC workflow
  • run roave/backward-compatibility-check without appending the removed flag

Verification

  • In yiisoft/validator, removed the caller input and verified vendor/bin/roave-backward-compatibility-check passes with a Roave baseline for the expected ignores.

Copilot AI review requested due to automatic review settings May 31, 2026 12:58
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 3a5486c5-b28e-44c3-a1dd-75ae2d21d7b1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-bc-install-development-dependencies

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the install-development-dependencies input from the reusable BC workflow and drops the conditional flag from the roave/backward-compatibility-check invocation.

Changes:

  • Remove the install-development-dependencies workflow input.
  • Simplify the BC check command to a static invocation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@samdark samdark merged commit ab62d6b into master May 31, 2026
4 checks passed
@samdark samdark deleted the remove-bc-install-development-dependencies branch May 31, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants