ci: add Windows to test matrix

Question

ci: add Windows to test matrix

lread opened this issue 2 years ago · 5 comments

In support of #18.

We currently only run our tests on ubuntu.
I'll add Windows into the mix.
We can add macOS at some later date if we see some reason to do so.

I'll also review CI config and update as appropriate, for example:

update outdated actions
add Clojure 1.11 to matrix
replace JDK 16 with JDK 17 in matrix

Answer 1 · 2022-08-12T18:39:11.000Z

I think that we can reasonably stop testing against Clojure 1.7 at this point.
Our jar references Clojure 1.8.0 which implies this our minimum Clojure version.

Answer 2 · 2022-08-12T18:41:48.000Z

@lread I suggest using neil add test for adding the cognitect test runner.

Answer 3 · 2022-08-12T18:42:31.000Z

Oh, wait, that's already been added.

Answer 4 · 2022-08-12T18:42:49.000Z

Yes, agreed on the above.

Answer 5 · 2022-08-12T18:43:52.000Z

Thanks for the feedback, it can get lonely sometimes! 🙂