diff options
author | Edward Z. Yang <ezyang@cs.stanford.edu> | 2014-08-09 02:05:53 +0100 |
---|---|---|
committer | Edward Z. Yang <ezyang@cs.stanford.edu> | 2014-08-10 20:22:56 +0100 |
commit | ba9277c72ab39bbd85ef52524151bf27a6c106f8 (patch) | |
tree | 726a8afbb2f1cf65cc5efc0483376d3e55622e26 /.arclint | |
parent | 326046739801a380c5457ef4c87bce8fb95497ba (diff) | |
download | haskell-ba9277c72ab39bbd85ef52524151bf27a6c106f8.tar.gz |
Tweak linting rules.
Summary:
- Removed the default text rule that was applied to all files; it
produced way too many spurious warnings. For now, text lint is applied
to C, Haskell and Shell. Add more if you care.
- Makefiles and shell scripts had their max line length bumped a healthy
amount.
A pile of files still fail to lint, so these might even still be too
aggressive.
Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
Test Plan: none
Reviewers: hvr, austin
Subscribers: simonmar, relrod, ezyang, carter
Differential Revision: https://phabricator.haskell.org/D131
Diffstat (limited to '.arclint')
-rw-r--r-- | .arclint | 18 |
1 files changed, 14 insertions, 4 deletions
@@ -12,24 +12,34 @@ "nolint": { "type": "nolint" }, - "text": { + "haskell": { "type": "text", - "exclude": [ "(\\.xml)", "(Makefile)", "(\\.mk)" ], + "include": ["(\\.(l?hs(-boot)?|x|y\\.pp)(\\.in)?$)"], "severity": { "5": "disabled" } }, + "c": { + "type": "text", + "include": ["(\\.(c|h)(\\.in)?$)"] + }, "text-xml": { "type": "text", - "include": "(\\.xml)", + "include": "(\\.xml$)", "severity": { "5": "disabled", "3": "disabled" } }, + "shell": { + "type": "text", + "include": [ "(\\.sh$)" ], + "text.max-line-length": 200 + }, "makefiles": { "type": "text", - "include": [ "(Makefile)", "(\\.mk)" ], + "include": [ "(Makefile$)", "(\\.mk$)" ], + "text.max-line-length": 200, "severity": { "2": "disabled" } |