We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simply requiring a namespace for a protocol to be implemented in a record causes a false positive when using the :unused-namespaces linter.
lein eastwood '{:linters [:unused-namespaces]}' == Eastwood 0.2.1 Clojure 1.6.0 JVM 1.7.0_51 Directories scanned for source files: src test == Linting protocol-test.foo == == Linting protocol-test.bar == Entering directory `/Users/derek/apps/protocol-test' src/protocol_test/bar.clj:1:1: unused-namespaces: Namespace protocol-test.foo is never used in protocol-test.bar == Warnings: 1 (not including reflection warnings) Exceptions thrown: 0 Subprocess failed
bar.clj
(ns protocol-test.bar (:require [protocol-test.foo :as foo])) (defrecord Bar [whatever] foo/Foo (foo [this] whatever))
foo.clj
(ns protocol-test.foo) (defprotocol Foo (foo [this]))
The text was updated successfully, but these errors were encountered:
Thanks for the small test case to reproduce the issue.
Sorry, something went wrong.
@jafingerhut ugh i was bit by this one too, any chance of getting this merged?
Any updates on it? @jafingerhut
This should be fixed in the forthcoming Eastwood 0.2.5 release.
Successfully merging a pull request may close this issue.
Simply requiring a namespace for a protocol to be implemented in a record causes a false positive when using the :unused-namespaces linter.
bar.clj
foo.clj
The text was updated successfully, but these errors were encountered: