On Thu, May 4, 2017 at 5:14 PM, Legoktm <legoktm.wikipedia(a)gmail.com> wrote:
If you encounter any bugs or have suggestions on new
rules, please reply
to this thread or file a bug in the #MediaWiki-Codesniffer Phabricator
project.
I ran this over a few repositories and ran into some issues.
- "@param string $name" seems fine on a method like
"getItemByName()",
there's little point in making that "@param string $name The name"
- It whines if PHP builtins aren't documented. Do we really need to
document __toString() over and over again?
- Doxygen is happy to inherit documentation from the parent class for an
overridden method. Your phpcs check doesn't even honor @inheritdoc, it want
everything copied.
At that point I gave up looking for wheat in the chaff.
--
Brad Jorsch (Anomie)
Senior Software Engineer
Wikimedia Foundation