This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-08-30
Channels
- # announcements (12)
- # babashka (25)
- # biff (30)
- # cherry (34)
- # cider (46)
- # clj-kondo (23)
- # clojure (37)
- # clojure-berlin (6)
- # clojure-europe (12)
- # clojure-nl (4)
- # clojure-norway (6)
- # clojure-uk (2)
- # clojurescript (8)
- # conjure (1)
- # cursive (4)
- # data-science (11)
- # datalevin (12)
- # datascript (15)
- # emacs (2)
- # events (1)
- # fulcro (14)
- # graalvm (16)
- # gratitude (23)
- # honeysql (11)
- # jobs (2)
- # jobs-discuss (14)
- # kaocha (1)
- # leiningen (8)
- # nbb (45)
- # off-topic (7)
- # portal (8)
- # re-frame (9)
- # releases (2)
- # shadow-cljs (24)
- # squint (5)
- # tools-build (17)
- # tools-deps (7)
- # vim (5)
nice, at Nubank we had some crashes in prod related to use schema.core/with-fn-validation
in a production code 😅 , so we are thinking on add a custom lint for that function to check if the filename contains a src
(usually, at least for our services, production code)
not namespaces, because we can't know only by a ns if that's is from test source-path
Wouldn't that just be something for: https://github.com/clj-kondo/clj-kondo/blob/master/doc/linters.md#discouraged-var
Actually, we do use it on tests, using that would make the warn for tests as well right? :/
yes, but you can make an ns-group for your tests (e.g. all namespaces ending with -test
) and then undo it for the tests
the ns-group :pattern
is a pattern for the ns right? I think that won't work since we can't know if a ns is a test, for example test/my_service/foo/my_feature.clj
(`(ns my-service.foo.my-feature)`)
yeah I know, but we have some integration legacy tests not using that convention, I think it'd be convenient to have a file-pattern
optional, WDYT?
@U04V15CAJ WDYT? would you accept a issue about it? I can try to find some time to implement the file-pattern support if you want