dtype-next provides config for nippy

[borkdude]

Hi @cnuernber!

Thanks for providing a clj-kondo config for your library users, much appreciated. I found one peculiarity today in someone's project:

In this particular case I stumbled upon an issue in someone's project and it was hard to track down where this came from since I didn't expect dtype next to provide a clj-kondo configuration for nippy.

Libraries should probably not provide configuration for other libraries that they don't own.

Thanks again,

Michiel

[borkdude]

@cnuernber In particular this construct caused the "unused value", since type is a "pure" function:

dtype-next/resources/clj-kondo.exports/cnuernber/dtype-next/taoensso/nippy.clj

Line 9 in f16f66f

(type ~dt)

The dt argument could probably also be ignored, right?

[cnuernber]

Both points I think are valid - the type there was a debug helper that didn't get fully removed. That being said I myself don't use any kondo-based tools so I have not been maintaining or testing this integration and no one else has stepped up to do so so I think there are two options really - fix just this with a PR or remove it.

[borkdude]

I think we can move the nippy config here:

https://github.com/clj-kondo/configs

which is a repo to host configs for third party libraries via dependencies that people can include in their deps.edn.

I'll add it over there and you can remove (only the nippy!) config here.

[cnuernber]

Great! Will do.

[borkdude]

Added the config: https://github.com/clj-kondo/configs/tree/main/configs/com.taoensso/nippy

[cnuernber]

That config is gone from this project :-)