search

Engelberg / instaparse

Eclipse Public License 1.0
2.74k stars 149 forks source link

Include clj-kondo config #231

Closed toniz4 closed 7 months ago

toniz4 commented 9 months ago

Some functions coming from instaparse.core are seen as unresolved by clj-kondo, like transform and span. This happens because clj-kondo doesn't expand macros, and since those symbols are defined with defclone, clj-kondo considers them undefined. It's a false negative, but It would be nice if we did not need to manually suppress those errors.

We would only need to add a config.edn file as described by the clj-kondo documentation

If it's something you wish to add, I will be happy to make a PR.

Engelberg commented 9 months ago

Yes, I'd be happy to include this, and the timing is great as I'm about to spend some time on instaparse pull requests. I haven't used clj-kondo myself, so if you're willing to make a PR, that would be appreciated.