Merge pull request #339 from crystal-ameba/Sija/tweak-ci

.github/workflows/ci.yml

@@ -2,7 +2,9 @@ name: CI
 
 on:
   push:
+    branches: [master]
   pull_request:
+    types: [opened, synchronize, reopened]
   schedule:
     - cron: "0 3 * * 1" # Every monday at 3 AM
 
@@ -28,7 +30,10 @@ jobs:
         run: shards install
 
       - name: Run specs
-        run: make test
+        run: crystal spec
 
-      - name: Check formatting
+      - name: Build ameba binary
-        run: crystal tool format --check
+        run: shards build -Dpreview_mt
+
+      - name: Run ameba linter
+        run: bin/ameba --all

Makefile

@@ -4,19 +4,35 @@ PREFIX ?= /usr/local
 SHARD_BIN ?= ../../bin
 CRFLAGS ?= -Dpreview_mt
 
-build: bin/ameba
+.PHONY: build
-bin/ameba:
+build:
 	$(SHARDS_BIN) build $(CRFLAGS)
+
+.PHONY: lint
+lint: build
+	./bin/ameba --all
+
+.PHONY: spec
+spec:
+	$(CRYSTAL_BIN) spec
+
+.PHONY: clean
 clean:
 	rm -f ./bin/ameba ./bin/ameba.dwarf
+
+.PHONY: install
 install: build
 	mkdir -p $(PREFIX)/bin
 	cp ./bin/ameba $(PREFIX)/bin
+
+.PHONY: bin
 bin: build
 	mkdir -p $(SHARD_BIN)
 	cp ./bin/ameba $(SHARD_BIN)
+
+.PHONY: run_file
 run_file:
 	cp -n ./bin/ameba.cr $(SHARD_BIN) || true
-test: build
+
-	$(CRYSTAL_BIN) spec
+.PHONY: test
-	./bin/ameba --all
+test: spec lint