Merge pull request #179 from humanmade/fix-docker-build

Fix builds / release github actions

.github/workflows/build-docker.yml

@@ -3,9 +3,9 @@ name: Docker Build
 on:
   push:
     branches:
-      - 'master'
+      - "**"
     tags:
-        - "*"
+      - "**"
 jobs:
   docker:
     runs-on: ubuntu-latest
@@ -14,11 +14,8 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 18.x
-      - uses: aws-actions/setup-sam@v2
-        with:
-          use-installer: true
       - run: npm install
-      - run: npm run build
+      - run: npx tsc
       - name: Docker meta
         id: meta
         uses: docker/metadata-action@v3
@@ -36,6 +33,7 @@ jobs:
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
       - name: Build and push latest
+        if: github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/')
         uses: docker/build-push-action@v2
         with:
           file: Dockerfile

.github/workflows/release.yml

@@ -3,7 +3,9 @@ name: Release
 on:
   push:
     tags:
-    - "*"
+    - "**"
+    branches:
+    - '**'
 
 jobs:
   build:
@@ -21,7 +23,12 @@ jobs:
       - run: npm run build
       - run: npm run build-zip
       - uses: softprops/action-gh-release@v1
+        if: github.event_name == 'pull_request'
         with:
           files: lambda.zip
           name: lambda
-
+      - uses: actions/upload-artifact@v3
+        if: github.event_name == 'push' && github.ref_type == 'tag'
+        with:
+          path: lambda.zip
+          name: lambda

.github/workflows/test.yml

@@ -1,7 +1,7 @@
 name: Test
 
 on:
-  pull_request:
+  push:
     branches:
       - '**'
 

tsconfig.json

@@ -3,7 +3,6 @@
 		"target": "es2020",
 		"strict": true,
 		"preserveConstEnums": true,
-		"noEmit": true,
 		"sourceMap": false,
 		"module": "ES2022",
 		"moduleResolution": "node",
@@ -13,6 +12,7 @@
 		"isolatedModules": true,
 		"types": ["node"],
 		"resolveJsonModule": true,
+		"outDir": "dist"
 	},
 	"exclude": ["node_modules"],
 	"include": ["./src/**/*.ts", "./*.d.ts", "./tests/**/*.ts"],