- Issue created by @catch
- 🇮🇹Italy mondrake 🇮🇹
I suggest to wait for conversions of annotations to attributes to be completed, then check on attributes instead. Probably a PHPStan rule should be written here.
- 🇮🇹Italy mondrake 🇮🇹
Changed mind and added a PHPStan rule checking for either annotations or attributes on abstract test base classes. We have 17. errors to be solved.
- 🇮🇹Italy mondrake 🇮🇹
Because fixing the errors that this rule surfaces is not trivial. So better leave that to follow ups IMHO, while the rule prevents adding more leaks.
- 🇮🇹Italy mondrake 🇮🇹
Well, the new rule introduces new errors, so either we fix them or we baseline them :)
- 🇺🇸United States smustgrave
Only other question is should annotations be attributes or does it not matter
- 🇮🇹Italy mondrake 🇮🇹
A check to prevent the use of annotation is already done by PHPUnit itself when it is configured to fail on PHPUnit deprecations. See 📌 Complete test annotations to attributes conversion for Drupal/Test/Component Active for an example.
IMHO we should not replicate such a check here.
- 🇺🇸United States smustgrave
In that case I'll go on a limb and say this one is probably ready.