search

moodlehq / moodle-cs

Moodle Coding Style
https://github.com/moodlehq/moodle-cs
GNU General Public License v3.0
18 stars 16 forks source link

Verify that all the "todo" comments have an MDL issue associated #90

Closed stronk7 closed 10 months ago

stronk7 commented 10 months ago

Commented recently in a chat, it would be interesting to start emitting a warning when any "todo" comment:

is missing any MDL issue associated.

Surely we could apply for the same in @deprecated phpdoc tags too.

Note that all them are part of the coding style:

Ciao :-)

marinaglancy commented 10 months ago

We were thinking of creating something like this for Workplace, but we will require the WP- prefix , not MDL-. I think a lot of plugin developers may need to customise this rule to have their own regex