- 🇬🇧United Kingdom catch
@param and @return for new code or any test being updated are now covered by type hinting.
This does not cover (sorry) @covers but I think we could allow only @covers in that case.
Helper methods - if they're on an individual test, no docs for me. If they're on a base test class or trait, they should have docs anyway.
- 🇬🇧United Kingdom catch
Added myself as a supporter and adjusted the proposed wording change in the issue summary.
- 🇳🇿New Zealand quietone
Move the current and proposed text to expected place in the issue summary.
I suggest the following.
All classes and all of their methods (including private methods but excluding constructors) must be documented. This does not apply to test methods on concrete test classes. Such methods should have a self-documenting method name. When the name is not sufficient documentation should be added.What sniffs would no longer be applied to tests?