Re:Disable comments make static analysis tools worse
by olliej on Tuesday July 06, 2021 @01:01AM
(#27744395)
one fun issue is what to do with test cases. I know of at least one project that uses precommit hooks to test with eslint (or something), etc but has tests that are explicitly checking for things that eslint disallows. Tests like that end up requiring disable comments by design.
(Also remember that time the webkit repo got hosed by someone committing the sha collision as a test case?)
(Also remember that time the webkit repo got hosed by someone committing the sha collision as a test case?)