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"],