This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-01-14
Channels
- # announcements (2)
- # aws (1)
- # babashka (18)
- # babashka-sci-dev (103)
- # beginners (165)
- # calva (51)
- # cider (8)
- # circleci (1)
- # clj-kondo (22)
- # clj-on-windows (2)
- # cljdoc (1)
- # cljfx (31)
- # cljs-dev (16)
- # clojure (81)
- # clojure-europe (71)
- # clojure-nl (7)
- # clojure-uk (11)
- # clojurescript (20)
- # code-reviews (26)
- # conjure (1)
- # contributions-welcome (1)
- # core-async (15)
- # cursive (8)
- # datomic (8)
- # defnpodcast (2)
- # eastwood (24)
- # emacs (10)
- # events (1)
- # fulcro (4)
- # funcool (31)
- # graalvm (43)
- # graphql (8)
- # honeysql (9)
- # introduce-yourself (1)
- # jobs (12)
- # kaocha (3)
- # lsp (28)
- # malli (4)
- # meander (4)
- # membrane (7)
- # off-topic (64)
- # other-languages (3)
- # pedestal (1)
- # polylith (31)
- # portal (5)
- # re-frame (4)
- # reitit (1)
- # releases (5)
- # rum (2)
- # schema (2)
- # sci (34)
- # shadow-cljs (21)
- # vscode (1)
Is it possible to turn off this check with some option? https://github.com/jonase/eastwood/blob/f6b278662e53dd738ca119822b888ff445890fa2/src/eastwood/error_messages.clj#L291
Why would you want to do so? If it's some sort of false positive, I'd rather have it fixed
I have a polylith code-structure and want to run eastwood in the root deps.edn
With zero config it should pick up the source-paths / test-paths correctly, out of the :paths
that the Clojure CLI computes.
(Make sure to use a recent-enough version of Eastwood)
> With zero config Related to that, if you have config that tries setting source-paths, I'd recommend removing it
yeah It is better to just use aliases with extra-paths I figured after your reply π
A question regarded your component definitions. @U45T93RA6 out of curiousity Do you have components like this: components/netty-http-client/src/org/http-client/interface.clj components/jetty-http-client/src/org/http-client/interface.clj so that you can swap out implementations via the project deps.edn?
We follow the poly patterns quite strictly with that "build-time impl swapping" in mind However I think haven't gotten to give use - no scenario that needed it has popped up
@U45T93RA6 a bit more context that might help you answer if itβs me or eastwood. im running eastwood in the top-level deps.edn in polylith.
{:main-opts ["-m"
"eastwood.lint"
;; Any Eastwood options can be passed here as edn:
]
:extra-deps {jonase/eastwood {:mvn/version "1.1.1"}}}
clj -M:dev:eastwood
Does the error message include Detected a mismatch between filenames and namespaces while
?
If so what does it say after while
?
It does not include this error. It includes errors like the one above and:
No other linting checks will be performed until these problems have
been corrected.
The 'should have namespace' and 'should be in file' messages above are
merely suggestions. It may be better in your case to rename both the file and namespace to avoid name collisions.
Another line is: maybe you have tools.namespace as part of the project, but it's badly set up. clojure.tools.namespace.repl/refresh-dirs
should evaluate to all your src
and test
paths
An alias which was being used had an extra-paths definition which caused this issue. This was pinpointed quite quickly after trying to run eastwood on another polylith repository.
The root β.β π