search

redplanetlabs / proxy-plus

A replacement for Clojure's proxy that's 10x faster and more usable
Apache License 2.0
175 stars 10 forks source link

Add support for clj-kondo #14

Closed ieugen closed 2 years ago

ieugen commented 2 years ago

I imported this library in my ocde and while working on it from Calva I noticed it does not have support for clj-kondo.

To my knowledge, macro support needs to be added for this to work. Perhaps of help: https://github.com/clj-kondo/clj-kondo/blob/master/doc/config.md#unrecognized-macros

nathanmarz commented 2 years ago

This is something you can add to your own clj-kondo config. It doesn't need to be part of this project.

ieugen commented 2 years ago

Thanks for the feedback. Wouldn't it be better (for reuse) to be part of this project?

nathanmarz commented 2 years ago

Yes, but it would also be something else we have to maintain.

ieugen commented 2 years ago

@nathanmarz :

It seems that :lint-as works pretty good at hiding nresolved symbol errors.

I used it like this (clj-kondo config as namespace metadata):

(ns dev.schema
  {:clj-kondo/config '{:lint-as {com.rpl.proxy-plus/proxy+ clojure.core/proxy}}}