Remove unnecessary testing and static analysis dependencies - #57
Open
fadrian06 wants to merge 1 commit into
Open
Remove unnecessary testing and static analysis dependencies#57fadrian06 wants to merge 1 commit into
fadrian06 wants to merge 1 commit into
Conversation
Remove Pest, PHP-CS-Fixer, and PHPStan from composer.json to streamline the development environment. These tools are no longer required for the project's current workflow.
Contributor
There was a problem hiding this comment.
馃煝 Approval recommended
The reviewed dependency cleanup is complete with no unresolved blocking issues.
Pull request overview
Removes redundant development dependencies and the unused PHPStan plugin configuration from composer.json.
Changes:
- Removes Pest, PHP CS Fixer, PHPStan, and the PHPStan extension installer.
- Removes the PHPStan plugin allowlist entry.
File summaries
| File | Description |
|---|---|
composer.json |
Simplifies development dependencies and plugin configuration. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
What kind of change does this PR introduce? (pls check at least one)
pestphp/pest,friendsofphp/php-cs-fixerare already required byleafs/billing.phpstan/phpstanis required byrector/rectorandphpstan/extension-installerwas not being usedDescription
This pull request simplifies the development dependencies in the
composer.jsonfile by removing several tools related to static analysis and code style, as well as updating the plugin configuration.Development dependency cleanup:
phpstan/phpstanandphpstan/extension-installerfrom therequire-devsection, which means PHPStan static analysis will no longer be available as a development dependency.friendsofphp/php-cs-fixerfrom therequire-devsection, so PHP CS Fixer for code style checks is no longer included.pestphp/pestfrom therequire-devsection, which means the Pest testing framework is no longer a development dependency.Configuration update:
allow-pluginssection to removephpstan/extension-installer, reflecting the removal of PHPStan-related dependencies.Does this PR introduce a breaking change? (check one)