Repository navigation
Check Composer dependencies in the test workflow - #133
Merged
shadetheory merged 1 commit intoOct 29, 2026
Merged
shadetheory merged 1 commit into
shadetheory merged 1 commit into
Conversation
Add the analyser as a dev dependency and run it after composer validate. Ignore only the false positive where laravel/framework replaces illuminate/*. Co-authored-by: Jordan Partridge <jordan@partridge.rocks>
shadetheory
merged commit Oct 29, 2026
38a3e34
into
cursor/declare-used-dependencies-8db1
23 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
shipmonk/composer-dependency-analyseras a dev dependency and run it in the test workflow, in its own step immediately aftercomposer validate.The test matrix rewrites
composer.jsonto pin Laravel and Testbench for that job. The publishedcomposer.jsonis restored before validation, so the analyser checks this package's own requirements.--no-check-lockis set because the lock file belongs to the matrix install, and this repository does not commit a lock file.composer-dependency-analyser.phpignores one false positive:orchestra/testbenchinstallslaravel/framework, and that package replacesilluminate/*. The analyser then attributes those classes tolaravel/frameworkand reports the Illuminate requirements as unused. The config ignores that shadow report and the unused report for the Illuminate packages this package requires. It does not ignore anything else.The config file is
export-ignore, like the other development config.This branch includes the dependency declaration changes from the other pull request.
Test plan
vendor/bin/composer-dependency-analyserreports no issuescomposer validate --strict --no-check-lockvendor/bin/pest