-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
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
Kondo linter tests for :metabase/validate-logging
#43104
Conversation
|
(Thread/sleep interval-ms) | ||
(Thread/sleep (long interval-ms)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed reflection warning introduced by #42316
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔥🔥🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@camsaul Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone? |
Introduce a way to run tests against Kondo linters.
Convert the test
comment
examples for our custom:metabase/validate-logging
linter to tests. This serves as an example of how to write tests for Kondo linters and a proof-of-concept.To make this work I moved
.clj-kondo/hooks
to.clj-kondo/src/hooks
; tests live in matching test namespace in.clj-kondo/test/hooks
.I added an alias so you can easily run Kondo linter tests with
clj -X:dev:test:test/kondo
.