Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions composer.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "saucebase/module-installer",
"description": "Custom Composer installer for Sauce Base modules (type: saucebase-module). Installs to Modules/<Name> with configurable base dir.",
"description": "Custom Composer installer for Sauce Base modules (type: saucebase-module). Installs to modules/<name> (lowercase) with configurable base dir.",
"type": "composer-plugin",
"license": "MIT",
"authors": [
Expand All @@ -12,8 +12,8 @@
"require": {
"php": "^8.4",
"composer-plugin-api": "^2.0",
"symfony/finder": "^8.0",
"symfony/process": "^8.0"
"symfony/finder": "^7.0|^8.0",
"symfony/process": "^7.0|^8.0"
Comment on lines +15 to +16

Copilot AI Apr 25, 2026

Copy link

Choose a reason for hiding this comment

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

composer.json requirements were changed, but composer.lock is still pinned to the old content-hash. Since CI runs composer validate, this will fail unless composer.lock is regenerated (e.g., composer update --lock or update these packages with --lock).

Copilot uses AI. Check for mistakes.
Comment on lines +15 to +16

Copilot AI Apr 25, 2026

Copy link

Choose a reason for hiding this comment

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

This PR claims Symfony 7 compatibility by broadening the constraint, but the committed lockfile (and CI) will still test against Symfony 8 only. Consider adding CI coverage for a Symfony 7 install (e.g., a job that updates symfony/finder + symfony/process to the latest 7.x allowed by the constraint and runs the PHPUnit suite) to ensure the new supported range is actually verified.

Copilot uses AI. Check for mistakes.
},
"require-dev": {
"composer/composer": "^2.0",
Expand Down
Loading