Skip to content

deps: use shipmonk/composer-dependency-analyser instead of icanhazstring/composer-unused and maglnet/composer-require-checker#9106

Merged
kubawerlos merged 4 commits intoPHP-CS-Fixer:masterfrom
6b7562617765726c6f73:dev_deps_2_to_1
Sep 30, 2025
Merged

deps: use shipmonk/composer-dependency-analyser instead of icanhazstring/composer-unused and maglnet/composer-require-checker#9106
kubawerlos merged 4 commits intoPHP-CS-Fixer:masterfrom
6b7562617765726c6f73:dev_deps_2_to_1

Conversation

@kubawerlos
Copy link
Copy Markdown
Member

@kubawerlos kubawerlos commented Sep 27, 2025

Replaces #9096

@kubawerlos kubawerlos added the dependencies Pull requests that update a dependency file label Sep 27, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Sep 27, 2025 •
edited
Loading

Coverage Status

coverage: 94.149% (+0.01%) from 94.139%
when pulling c4084b2 on 6b7562617765726c6f73:dev_deps_2_to_1
into bcf975b on PHP-CS-Fixer:master.

@kubawerlos kubawerlos mentioned this pull request Sep 28, 2025
Closed
keradus
keradus reviewed Sep 30, 2025
View reviewed changes
Comment thread composer.json
Wirone
Wirone approved these changes Sep 30, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@Wirone Wirone left a comment

Choose a reason for hiding this comment

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

Shipmonk influence 😅.

@Wirone
Copy link
Copy Markdown
Member

Wirone commented Sep 30, 2025

@kubawerlos just to be sure: does Shipmonk's tool cover both composer.jsons (regular and dev-tools)?

@kubawerlos
Copy link
Copy Markdown
Member Author

kubawerlos commented Sep 30, 2025

@kubawerlos just to be sure: does Shipmonk's tool cover both composer.jsons (regular and dev-tools)?

No, as all those tools, only the one in the root. We can try to cover the one in dev-tools, but I imagine it would mostly be the exceptions - after all, it checks the PHP code using deps, and dev tools are mostly CLI tools with config.s

@kubawerlos kubawerlos merged commit e0516ca into PHP-CS-Fixer:master Sep 30, 2025
32 of 33 checks passed
@kubawerlos kubawerlos deleted the dev_deps_2_to_1 branch September 30, 2025 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Wirone Wirone Wirone approved these changes

@keradus keradus keradus approved these changes

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kubawerlos @coveralls @Wirone @keradus