We should run the style checker over python files in LayoutTests.
<rdar://problem/76122813>
Created attachment 424951 [details] Patch
Comment on attachment 424951 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=424951&action=review > Tools/Scripts/webkitpy/style/checker.py:640 > - elif basename in _NEVER_SKIPPED_FILES: > - return False > + for suffix in _NEVER_SKIPPED_FILES: > + if basename.endswith(suffix): > + return False I saw some potential behavior change: `A-TestExpectations` will not return False and now it will?
Comment on attachment 424951 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=424951&action=review >> Tools/Scripts/webkitpy/style/checker.py:640 >> + return False > > I saw some potential behavior change: `A-TestExpectations` will not return False and now it will? Yes, although that may not be a bad thing. First, there isn't anything like "*TestExpectations" in LayoutTests. But more importantly, if we do end up picking up a file we don't expect to, we just end up running the style checker more aggressively than we intended to, which is unlikely to be harmful
Comment on attachment 424951 [details] Patch r=me
Committed r275444: <https://commits.webkit.org/r275444> All reviewed patches have been landed. Closing bug and clearing flags on attachment 424951 [details].