moodle-cs icon indicating copy to clipboard operation
moodle-cs copied to clipboard

Make the TodoComment Sniff to require a complete regex (start, end delimiters)

Open stronk7 opened this issue 10 months ago • 0 comments

While working on a new release of the moodle-plugin-ci including the latest version of moodle-cs, it has been detected that there is a small discordance between the TodoComment sniff (and its commentRequiredRegex property/config) and the FileExpectedTags (and its preferredLicenseRegexproperty/config):

  • The TodoComment ones don't require the usual regex delimiters and adds ~ automatically.
  • The FileExpectedTags ones do require a complete regex with delimiters.

I think that the later is better, so this issue is about to change the former to also require a complete regex with delimiters.

stronk7 avatar Mar 29 '24 18:03 stronk7