Page MenuHomePhabricator

Don't require documentation for __toString()
Closed, ResolvedPublic

Description

Pretty obvious what it does.

Event Timeline

Change 352361 had a related patch set uploaded (by Legoktm; owner: Legoktm):
[mediawiki/tools/codesniffer@master] Don't require documentation for '__toString'

https://gerrit.wikimedia.org/r/352361

Possibly also __destruct(), __sleep(), __wakeup(), and __clone().

__construct() only needs documentation of its parameters, if there aren't any there's probably nothing worth documenting there.

__call(), __callStatic(), __get(), __set(), __isset(), __unset(), __invoke(), and __set_state() probably do want documentation as to what exactly they're doing.

Change 352361 merged by jenkins-bot:
[mediawiki/tools/codesniffer@master] Don't require documentation for '__toString'

https://gerrit.wikimedia.org/r/352361