Merge pull request #215 from tylerslaton/linter

feat: add better linting targets and CI

.github/workflows/admin.yml

@@ -0,0 +1,32 @@
+name: admin
+
+on:
+  pull_request:
+    branches:
+      - main
+    paths:
+      - ui/admin/**
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20.17.0"
+
+      - name: Install dependencies
+        run: |
+          cd ui/admin
+          npm install
+
+      - name: Run linter
+        run: make lint-admin
+
+      - name: Verify no changes
+        run: make no-changes

Makefile

@@ -23,4 +23,20 @@ dev: ui
 	@echo "Starting dev otto server and admin UI..."
 	./dev.sh
 
-.PHONY: ui build all clean dev
+# Lint the project
+lint: lint-admin
+
+lint-admin:
+	cd ui/admin && \
+	npm run format && \
+	npm run lint
+
+no-changes:
+	@if [ -n "$$(git status --porcelain)" ]; then \
+		git status --porcelain; \
+		git --no-pager diff; \
+		echo "Encountered dirty repo!"; \
+		exit 1; \
+	fi
+
+.PHONY: ui build all clean dev lint lint-admin lint-api no-changes fmt tidy