Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

exclude-linters option doesn't seem to work #7

Closed
christopheryoung opened this Issue May 10, 2012 · 4 comments

Comments

Projects
None yet
2 participants

Using Eastwood 0.0.2, Clojure 1.4 and lein2.

lein eastwood '{:exclude-linters [:misplaced-docstring]}'

at the command line

produces exactly the same output as lein eastwood (with tons of misplaced-docstring warnings).

Same thing when I added:

{:eastwood {:exclude-linters [:misplaced-docstring]}}

to my profiles.clj.

Am I doing anything obviously stupid?

Owner

jonase commented May 11, 2012

The name of the linter is :misplaced-docstrings

@jonase jonase closed this May 11, 2012

How embarrassing. Sorry for the bad report.

Ah, one small point: Is the "s" supposed to be missing in the :misplaced-docstring keyword on line 37 of linters/misc.clj? The report produced tells you that the linter is :misplaced-docstring instead of :misplaced-docstrings, which is what led me to miss that "s" in spite of looking at the source.

@jonase jonase added a commit that referenced this issue May 11, 2012

@jonase jonase Fixed linter names, see #7 148dd4b
Owner

jonase commented May 11, 2012

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment