-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: add "force" option to PhpUnitDataProviderStaticFixer #6757
feature: add "force" option to PhpUnitDataProviderStaticFixer #6757
Conversation
Pull Request Test Coverage Report for Build 4520417842
💛 - Coveralls |
Rule sets | ||
--------- | ||
|
||
The rule is part of the following rule set: | ||
|
||
@PHPUnit100Migration:risky | ||
Using the `@PHPUnit100Migration:risky <./../../ruleSets/PHPUnit100MigrationRisky.rst>`_ rule set will enable the ``php_unit_data_provider_static`` rule. | ||
Using the `@PHPUnit100Migration:risky <./../../ruleSets/PHPUnit100MigrationRisky.rst>`_ rule set will enable the ``php_unit_data_provider_static`` rule with the default config. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we can have the new option used in some ruleset. I have a feeling that if we do not, the ppl will not be aware about this option and not using it.
[no good answer to this, myself]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's for a start do it in @PhpCsFixer:risky
, but in separate PR as there are quite few changes (including sorting of function that makes diff big).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding force
option to @PHPUnit100Migration:risky
might have more sense as without it the migration might not be successful (with it also as it will break the test suite, but will make it closer to successful migration).
Thank you @kubawerlos 🍻 |
Follow up to #6696 to help fully migrate to static data providers.