Submitter | Pierre-Yves David |
---|---|
Date | Aug. 6, 2016, 12:20 a.m. |
Message ID | <39b9df95-a5ca-788a-9821-579783586ab6@ens-lyon.org> |
Download | mbox | patch |
Permalink | /patch/16147/ |
State | Accepted |
Headers | show |
Comments
Patch
diff -r 4de25e275c00 contrib/check-commit --- a/contrib/check-commit Fri Aug 05 17:27:51 2016 -0400 +++ b/contrib/check-commit Sat Aug 06 02:16:06 2016 +0200 @@ -41,6 +41,10 @@ errors = [ (afterheader + r".{79,}", "summary line too long (limit is 78)"), (r"\n\+\n( |\+)\n", "adds double empty line"), (r"\n \n\+\n", "adds double empty line"), + # Forbid "_" in function name. + # + # We skip the check for cffi related functions. They use names mapping the + # name of the C function. C function names may contain "_". (r"\n\+[ \t]+def (?!cffi)[a-z]+_[a-z]", "adds a function with foo_bar naming"),