nextjournal / clerk

⚡️ Moldable Live Programming for Clojure
https://clerk.vision
ISC License
1.74k stars 75 forks source link

Remove unused namespace and variable for clj-kondo lint code #636

Open vedang opened 4 months ago

vedang commented 4 months ago

This is a minor fix, but it keeps showing up as a warning in my projects when I run clj-kondo from the command line, and I'd like to fix it :)

Steps to reproduce the problem:

  1. Run clj-kondo --lint "$(clojure -A:your:aliases -Spath)" --copy-configs --skip-lint to install clj-kondo configuration for your deps in some project that also uses clerk.

  2. Run clj-kondo --lint . in the project

The output contains:

./.clj-kondo/nextjournal/clerk/nextjournal/clerk/viewer.clj_kondo:3:14: warning: namespace clojure.string is required but never used
./.clj-kondo/nextjournal/clerk/nextjournal/clerk/viewer.clj_kondo:5:33: warning: unused binding lang
./.clj-kondo/nextjournal/clerk/nextjournal/clerk/viewer.clj_kondo:5:43: warning: unused binding opts

After this commit, the warnings go away