search

arxanas / lint381

lint381 checks your EECS 381 projects for coding style errors
19 stars 6 forks source link

more than 3 *** still matches the triple asterisc rule #7

Closed DavidSnider closed 8 years ago

DavidSnider commented 8 years ago

/***** Island Class *****/ would be flagged.

arxanas commented 8 years ago

I believe I considered matching the regex \b\*\*\*\b and it did not work for some comments for some reason, but I can check again.

arxanas commented 8 years ago

Well literally \b doesn't work because *** is not a word according to the regex rules and the things around it are therefore not word boundaries.

I didn't find any false negatives in the project 2 or 4 starter files that would occur due to the proposed change. You should be able to update to 1.3.7 once it goes through CI.