- π¬π§United Kingdom jonathan1055
This coding standards issue is discussing the potential changes:
#3444003: Change sniff rule for @todo to be capitalized and sniff non-php files β
There was discussion around improving the linting when we tried to enforce a previous attempt at @todo linting on core.
https://git.drupalcode.org/project/drupal/-/merge_requests/504#note_18998
Enforce additional rules around the formatting of @todo statements.
Decide on the exact rules we want to enforce around @todo statement formatting.
@todo \Drupal\foo\Bar should be valid.
@todo $foo variable name is valid.
@todo this should be flagged as a warning with a suggestion.
Consider if existing comment style sniffs cover this or should cover this vs adding logic to the @todo specific sniff.
Yes, in response to @klausi's points above, I think the logic could be reverted to allow any non-space except "-" or ":" as we had before. Then we give a separate warning if the first letter is a lower-case alpha and provide the fix to convert to upper case.
β https://github.com/pfrenssen/coder/pull/174#issuecomment-1252249092
Work off of this existing draft PR: https://github.com/pfrenssen/coder/pull/174
N/A
N/A
N/A
Active
8.3
Coder Sniffer
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This coding standards issue is discussing the potential changes:
#3444003: Change sniff rule for @todo to be capitalized and sniff non-php files β