Question: ALE-Kondo-Integration: Was Kondo always non-destructive/invasive against the source files #1668
-
ALE lints code inside vim. There is integration for kondo already. Kondo v2022.04.08 re-introduced Back when it was introduced first it led to problems with ALE, which I addressed with a PR against ALE but then redrew due to the original problem being gone (for now) in Kondo and concerns of the author. The current ALE-Kondo-integration uses a copy of the current source file (under My PR changed this to just use the actual source file (no copy). The ALE authors concern back then was, that changing this might break integration with old Kondo versions, that relied on the copy. To my knowledge Kondo always has been used to just read source ideally inside the project for the most insight and then only report it's findings. It was never destructive/invasive. Is this correct? Thanks for any hints. |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 2 replies
-
Perhaps @lispyclouds knows something about this? |
Beta Was this translation helpful? Give feedback.
-
I think I know now why the "temp approach" was chosen and is "better". This also works while the file is edited or a new file is not yet written. With my proposed ALE change this would only allow the linting to take place after saving the file, which would be less responsive for the developer. The question itself remains, but I think this has to be solved with a different approach with ALE. |
Beta Was this translation helpful? Give feedback.
-
@christoph-frick yes I don't think clj-kondo does any modifications to the file(s) its linting and it never did; its only concerned about reporting the findings (@borkdude can maybe confirm?) . Modifications via clj-kondo may be done by tooling that embeds it like clojure-lsp. |
Beta Was this translation helpful? Give feedback.
@christoph-frick yes I don't think clj-kondo does any modifications to the file(s) its linting and it never did; its only concerned about reporting the findings (@borkdude can maybe confirm?) . Modifications via clj-kondo may be done by tooling that embeds it like clojure-lsp.