search

brandonbloom / fipp

Fast Idiomatic Pretty Printer for Clojure
525 stars 44 forks source link

consider automating tests on circleci #68

Closed lread closed 4 years ago

lread commented 4 years ago

Hello!

Before working on #66, I found #67 and this made me wonder if you'd be interested in automating the running of tests on circleci.

As part of this work I propose:

  1. integrating cljs tests into lein via lein-cljsbuild.
  2. creating a circleci config that would automatically run tests on commit.

I don't know that automatically running the benchmark would be of interest/value, I guess it would test that it still works.

lread commented 4 years ago

Also to consider for a test runner is kaocha. Note though that it has a minimum requirement of clojure 1.9 and I guess that would be a showstopper for you if you want to still want to support clojure 1.8.

brandonbloom commented 4 years ago

I'm not interested in adding any more tooling for a project this small and infrequently updated. Thanks though.

lread commented 4 years ago

Fair enough! I totally understand one person's gift can be another person's burden and respect your choice.