Configure CI (#46)

- Configure clj-kondo, cljfmt
- Workflow status badge for the readme

Co-authored-by: Akshay Gupta <kitallis@gmail.com>

Author: scimetfoo

.clj-kondo/config.edn

@@ -0,0 +1 @@
+{:lint-as {honeysql.helpers/defhelper clj-kondo.lint-as/def-catch-all}}

.github/workflows/ci.yaml

@@ -0,0 +1,36 @@
+name: CI
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        java: [ '8', '11', '14' ]
+    name: For (java ${{ matrix.java }})
+    steps:
+      - uses: actions/checkout@v2
+      - name: Setup Java
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{ matrix.java }}
+      - name: Setup Clojure
+        uses: DeLaGuardo/setup-clojure@master
+        with:
+          tools-deps: '1.10.1.754'
+      - name: Run tests (core)
+        run: clojure -M:test:runner
+      - name: Run tests (clojurescript)
+        run: clojure -M:test:cljs-runner
+      - name: Install clj-kondo
+        run: |
+          curl -sLO https://raw.githubusercontent.com/borkdude/clj-kondo/master/script/install-clj-kondo
+          chmod +x install-clj-kondo
+          sudo ./install-clj-kondo
+      - name: Lint src
+        run: clj-kondo --lint src/
+      - name: Lint test
+        run: clj-kondo --lint test/
+      - name: Run cljfmt check
+        run: clojure -Sdeps '{:deps {cljfmt {:mvn/version "0.7.0"}}}' -m cljfmt.main check

.gitignore

@@ -8,3 +8,6 @@ pom.xml.asc
 /.nrepl-port
 .hgignore
 .hg/
+.clj-kondo/.cache
+.lsp/
+cljs-test-runner-out/

README.md

@@ -1,5 +1,4 @@
-# honeysql-postgres
-[![Clojars Project](https://img.shields.io/clojars/v/nilenso/honeysql-postgres.svg)](https://clojars.org/nilenso/honeysql-postgres) [![NPM Version](https://img.shields.io/npm/v/@honeysql/honeysql-postgres.svg)](https://www.npmjs.org/package/@honeysql/honeysql-postgres) [![cljdoc badge](https://cljdoc.org/badge/nilenso/honeysql-postgres)](https://cljdoc.org/d/nilenso/honeysql-postgres/CURRENT)
+# honeysql-postgres [![Actions Status](https://github.com/nilenso/honeysql-postgres/workflows/CI/badge.svg)](https://github.com/nilenso/honeysql-postgres/actions) [![Clojars Project](https://img.shields.io/clojars/v/nilenso/honeysql-postgres.svg)](https://clojars.org/nilenso/honeysql-postgres) [![NPM Version](https://img.shields.io/npm/v/@honeysql/honeysql-postgres.svg)](https://www.npmjs.org/package/@honeysql/honeysql-postgres) [![cljdoc badge](https://cljdoc.org/badge/nilenso/honeysql-postgres)](https://cljdoc.org/d/nilenso/honeysql-postgres/CURRENT)
 
 PostgreSQL extensions for the widely used [honeysql](https://github.com/jkk/honeysql).
 

src/honeysql_postgres/format.cljc

@@ -124,8 +124,7 @@
            (str " WHERE " (sqlf/format-predicate* where))))))
 
 (defn- format-upsert-clause [upsert]
-  (let [ks (keys upsert)]
-    (map #(format-clause % (find upsert %)) upsert)))
+  (map #(format-clause % (find upsert %)) upsert))
 
 (defmethod format-clause :upsert [[_ upsert] _]
   (sqlf/space-join (format-upsert-clause upsert)))
@@ -174,7 +173,7 @@
   (str
    ;; if the select clause has any columns in it then add a comma before the
    ;; window functions
-   (if (seq (:select complete-sql-map)) ", ")
+   (when (seq (:select complete-sql-map)) ", ")
    (->> fields
         (map format-over-clause)
         sqlf/comma-join)))
@@ -232,7 +231,7 @@
            util/get-first
            sqlf/to-sql)))
 
-(defmethod format-clause :create-extension [[_ [ extension-name if-not-exists]] _]
+(defmethod format-clause :create-extension [[_ [extension-name if-not-exists]] _]
   (str "CREATE EXTENSION "
        (when if-not-exists "IF NOT EXISTS ")
        (-> extension-name

src/honeysql_postgres/helpers.cljc

@@ -3,10 +3,11 @@
   (:require [honeysql-postgres.format]
             [honeysql.helpers :as sqlh #?(:clj :refer :cljs :refer-macros) [defhelper]]))
 
-;; Extension of the honeysql helper funcitons for postgreSQL
+;; Extension of the honeysql helper functions for postgreSQL
 
-(defn do-nothing [m]
+(defn do-nothing
   "Accepts a map and append {:do-nothing []} to it"
+  [m]
   (assoc m :do-nothing []))
 
 (defhelper do-update-set [m args]