summaryrefslogtreecommitdiff
path: root/.arclint
Commit message (Collapse)AuthorAgeFilesLines
* Remove arc scriptsÖmer Sinan Ağacan2019-02-191-125/+0
|
* Use python3 for lintersMatthew Pickering2016-12-231-3/+3
| | | | | | | | | | | | | | We now require python3 for the testsuite so rather than require two versions of python it makes sense to use python3 for the linters as well. Reviewers: austin, bgamari Reviewed By: bgamari Subscribers: thomie Differential Revision: https://phabricator.haskell.org/D2888
* arclint: Lint cabal filesBen Gamari2016-12-081-0/+16
| | | | | | | | | | Test Plan: Try linting a cabal file with trailing whitespace Reviewers: austin Subscribers: thomie, alanz Differential Revision: https://phabricator.haskell.org/D2799
* Testsuite: don't use --interactive in MakefilesThomas Miedema2016-05-271-0/+5
| | | | | | | | | | | | | | Add a linter to encourage the use of `$(TEST_HC_OPTS_INTERACTIVE)` instead of `$(TEST_HC_OPTS) --interactive -ignore-dot-ghci -v0`. It's too easy to forget one of those flags when adding a new test. Update submodule hpc. Reviewed by: austin Differential Revision: https://phabricator.haskell.org/D2265 GHC Trac Issues: #11468
* Reenable external-json lintersBen Gamari2016-03-291-0/+9
| | | | | | | | | | | | | | | | | | | | This is to be merged once a fix for the Harbormaster build script has been applied. This reverts commit a14296c2660521db8ba965065687e45cee4e3401. These were previously disabled due to Harbormaster validation failures. Also, apply CPP check to .c and .h files in addition to Haskell sources. Test Plan: Let Harbormaster validate Reviewers: austin Reviewed By: austin Subscribers: hvr, thomie Differential Revision: https://phabricator.haskell.org/D1580
* Temporarily disable external-json lintersHerbert Valerio Riedel2015-12-051-9/+0
| | | | | | | | | | | This temporarily disables the external json linters introduced via 020375d1e723339a95b86d0d3b8a8214b1cc144a and 901cab10609dc9795e57163834512373530fc4a5. The new linters break the differential validation in Harbormaster because the submodules are not cloned early enough (i.e. before `arc` is called by the script).
* lint: Add linter to catch uses of ASSERT macro that Clang dislikesBen Gamari2015-12-041-0/+5
| | | | | | In particular Clang rejects uses of CPP macros where the argument list is separated by a space from the macro name. Warn when we see ASSERT used in this way.
* Add linter to check for binaries accidentally added to repositoryBen Gamari2015-12-041-0/+4
| | | | | | | | | | | | | | | | This should catch mistakes like a703fbce20969e6f02e74fee76c0a9e22b513426. Adds an arcanist-external-json-linter submodule, which should eventually be mirrored on haskell.org resources. Test Plan: Validate Reviewers: thomie, hvr, austin Reviewed By: austin Differential Revision: https://phabricator.haskell.org/D1515
* Fix broken .arclintBartosz Nitka2015-10-241-1/+1
| | | | | | | | | | | | Test Plan: the linter works now Reviewers: ezyang, austin, bgamari Reviewed By: bgamari Subscribers: thomie Differential Revision: https://phabricator.haskell.org/D1359
* arclint: ReST doesn't need ArcanistMergeConflictLinterBen Gamari2015-10-231-1/+2
| | | | The section heading syntax trips up this linter. Fixes #10979.
* Cabal submodule update: hole support and tests.Edward Z. Yang2014-11-261-1/+1
| | | | | | | | | | | | | | Depends on D485 Summary: Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu> Test Plan: validate Reviewers: simonpj, austin Subscribers: thomie, carter Differential Revision: https://phabricator.haskell.org/D487
* arclint: Don't complain about tabs unless it's inside the diff.Edward Z. Yang2014-10-021-4/+12
| | | | | | | | | | | | Summary: Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu> Test Plan: none Reviewers: austin Subscribers: simonmar, ezyang, carter, thomie Differential Revision: https://phabricator.haskell.org/D303
* Tweak linting rules.Edward Z. Yang2014-08-101-4/+14
| | | | | | | | | | | | | | | | | | | | | | | 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
* arclint: update linting configurationAustin Seipp2014-07-211-3/+48
| | | | Signed-off-by: Austin Seipp <austin@well-typed.com>
* arclint: update rules for xml filesAustin Seipp2014-06-261-0/+9
| | | | | | | As pointed out by Edward, it's probably unreasonable to have the xml docs adhere to 80 columns, so we'll separate out the linters for now. Signed-off-by: Austin Seipp <austin@well-typed.com>
* arclint: disable Bad Charset lint ruleAustin Seipp2014-06-231-1/+4
| | | | Signed-off-by: Austin Seipp <austin@well-typed.com>
* Add .arclint fileAustin Seipp2014-06-091-0/+19
Signed-off-by: Austin Seipp <austin@well-typed.com>