Skip to content

Update composer.json require-dev to match integration test matrix#154

Merged
JanTvrdik merged 4 commits intomasterfrom
cleanup/update-composer-require-dev
Feb 25, 2026
Merged

Update composer.json require-dev to match integration test matrix#154
JanTvrdik merged 4 commits intomasterfrom
cleanup/update-composer-require-dev

Conversation

@JanTvrdik
Copy link
Copy Markdown
Member

Summary

  • Removed outdated extra.branch-alias (dev-master: 3.1-dev)
  • Updated require-dev version constraints to reflect what is actually tested in tests/matrix/**/*.sh
  • Switched deprecated single | separators to ||
  • Added doctrine/doctrine-bundle and symfony/yaml (present in matrix, missing from require-dev)
  • Dropped untested version ranges (e.g., Symfony 2.x/3.x, doctrine/cache 2.0)
  • Fixed minimum versions to match matrix (e.g., doctrine/orm ^2.7, nette/di ~2.4, nextras/dbal 5.0 stable)

Copilot AI review requested due to automatic review settings February 25, 2026 18:57
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

This PR synchronizes the composer.json development dependencies with the versions actually tested in the integration test matrix, improving consistency between declared dependencies and test coverage.

Changes:

  • Updated version constraint syntax from deprecated single | to || separators
  • Removed untested dependency versions (Symfony 2.x/3.x, doctrine/cache 2.0, dev versions)
  • Added missing dependencies present in test matrix (doctrine/doctrine-bundle, symfony/yaml)
  • Removed outdated extra.branch-alias configuration

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

@JanTvrdik JanTvrdik merged commit 01a33fe into master Feb 25, 2026
10 checks passed
@JanTvrdik JanTvrdik deleted the cleanup/update-composer-require-dev branch February 25, 2026 19:07
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.

2 participants