Add test hooks with try-repo

Also simplifies the optional a match a bit.

Fixes #15

.drone.yml

@@ -9,9 +9,8 @@ workspace:
 steps:
 
   - name: test
-    image: iamthefij/drone-pre-commit:latest
+    image: iamthefij/drone-pre-commit@sha256:6ed8dae6b0067bd2e145e36421bcfbbc68975ff7ddaa5a3f285b5bcdaf0989c8
     commands:
-      - pip install docker-compose
       - make all
 
   - name: notify

.pre-commit-hooks.yaml

@@ -1,19 +1,7 @@
 ---
 - id: docker-compose-check
-  name: Validate docker-compose files
-  description: Checks that docker-compose files are valid
+  name: Validate docker compose files
+  description: Checks that docker compose files are valid
   language: script
   entry: compose-check.sh
-  files: docker-compose.y[a]{0,1}ml$
-- id: hadolint
-  name: Lint Dockerfiles
-  description: Runs hadolint Docker image to lint Dockerfiles
-  language: docker_image
-  entry: hadolint/hadolint hadolint
-  files: Dockerfile
-- id: hadolint-system
-  name: Lint Dockerfiles
-  description: Runs system hadolint to lint Dockerfiles
-  language: system
-  entry: hadolint
-  files: Dockerfile
+  files: (docker-)?compose\.ya?ml$

Makefile

@@ -2,7 +2,7 @@
 
 all: check test
 
-test: test-negative test-positive
+test: test-negative test-positive test-hooks
 
 .PHONY: test-positive
 test-positive:
@@ -16,6 +16,10 @@ test-negative:
 	@echo "Check multiple files. Should error."
 	./compose-check.sh tests/docker-compose* && { echo 'fail'; exit 1; } || echo 'ok'
 
+.PHONY: test-hooks
+test-hooks:
+	pre-commit try-repo . --all-files
+
 # Installs pre-commit hooks
 .PHONY: install-hooks
 install-hooks:

README.md

@@ -2,13 +2,21 @@
 
 A set of [pre-commit](http://pre-commit.com) hooks for Docker services
 
+# Installation
+
+Add the following to your `.pre-commit-config.yaml` file
+
+```yaml
+  - repo: https://github.com/iamthefij/docker-pre-commit
+    rev: master
+    hooks:
+      - id: docker-compose-check
+```
+
+and then run `pre-commit autoupdate`.
+
+
 ## Hooks
 
 ### docker-compose-check
-Verifies that docker-compose files are valid by using `docker-compose config` to parse them.
-
-### hadolint
-Uses the [hadolint Docker image](https://hub.docker.com/r/hadolint/hadolint) to lint Dockerfiles.
-
-### hadolint-system
-Uses the whatever version of hadolint that you have installed to lint Dockerfiles. This requires you to have hadolint installed somewhere in your path.
+Verifies that docker compose files are valid by using `docker compose config` to parse them.

compose-check.sh

@@ -1,10 +1,19 @@
-#! /bin/bash
+#! /usr/bin/env bash
 # Verifies that files passed in are valid for docker-compose
 set -e
 
+if command -v docker &> /dev/null && docker help compose &> /dev/null; then
+    COMPOSE="docker compose"
+elif command -v docker-compose &> /dev/null ; then
+    COMPOSE=docker-compose
+else
+    echo "ERROR: Neither 'docker-compose' or 'docker compose' were found"
+    exit 1
+fi
+
 check_file() {
     local file=$1
-    docker-compose --file "$file" config --quiet 2>&1 \
+    env $COMPOSE --file "$file" config --quiet 2>&1 \
         | sed "/variable is not set. Defaulting/d"
     return "${PIPESTATUS[0]}"
 }