Addresses #331 by changing some CI stuff

* Updated Eastwood to 0.5.1
* Added set -Eeo pipefail to run-tests.sh (I don't like -x)
* Run CI on PR
* Add shellcheck to steps

I'm not going to run Eastwood on tests at this point (see my
comments in #334 for reasons).
This commit is contained in:
Sean Corfield 2021-06-21 15:16:18 -07:00
parent 13b640d486
commit 8ec08ca686
3 changed files with 5 additions and 2 deletions

View file

@ -1,6 +1,6 @@
name: Clojure CI name: Clojure CI
on: [push] on: [push, pull_request]
jobs: jobs:
build: build:
@ -18,6 +18,8 @@ jobs:
uses: DeLaGuardo/setup-clojure@master uses: DeLaGuardo/setup-clojure@master
with: with:
tools-deps: '1.10.3.839' tools-deps: '1.10.3.839'
- name: Lint test script
run: shellcheck run-tests.sh
- name: Run Tests - name: Run Tests
run: sh run-tests.sh all run: sh run-tests.sh all
- name: Check cljdoc.edn - name: Check cljdoc.edn

View file

@ -15,7 +15,7 @@
:main-opts ["-m" "cljs-test-runner.main"]} :main-opts ["-m" "cljs-test-runner.main"]}
:readme {:extra-deps {seancorfield/readme {:mvn/version "1.0.16"}} :readme {:extra-deps {seancorfield/readme {:mvn/version "1.0.16"}}
:main-opts ["-m" "seancorfield.readme"]} :main-opts ["-m" "seancorfield.readme"]}
:eastwood {:extra-deps {jonase/eastwood {:mvn/version "0.4.2"}} :eastwood {:extra-deps {jonase/eastwood {:mvn/version "0.5.1"}}
:main-opts ["-m" "eastwood.lint" "{:source-paths,[\"src\"]}"]} :main-opts ["-m" "eastwood.lint" "{:source-paths,[\"src\"]}"]}
:jar {:replace-deps {com.github.seancorfield/depstar {:mvn/version "2.0.216"}} :jar {:replace-deps {com.github.seancorfield/depstar {:mvn/version "2.0.216"}}
:exec-fn hf.depstar/jar :exec-fn hf.depstar/jar

View file

@ -1,4 +1,5 @@
#!/bin/sh #!/bin/sh
set -Eeo pipefail
echo ==== Test README.md ==== && clojure -M:readme && \ echo ==== Test README.md ==== && clojure -M:readme && \
echo ==== Lint Source ==== && clojure -M:eastwood && \ echo ==== Lint Source ==== && clojure -M:eastwood && \