Skip to content

Commit

Permalink
Fix workflows (#136)
Browse files Browse the repository at this point in the history
* Update workflows for main and staging branches

* Update deployment config for workflows

* Update staging-migrations workflow options

* Update workflow names for clarity
  • Loading branch information
sjoerdbeentjes authored Oct 4, 2024
1 parent 401d8ad commit 7cd62de
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/main-migrations.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,11 @@
name: Deploy production
name: Deploy to production

on:
pull_request:
types: [closed]
push:
branches:
- main

jobs:
apply-main-migrations:
Expand Down Expand Up @@ -44,6 +47,7 @@ jobs:
run: npm run migrations:apply-main

prepare:
needs: apply-main-migrations
runs-on: ubuntu-latest
outputs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
Expand Down Expand Up @@ -105,7 +109,6 @@ jobs:
publish-dir: "./dist"
production-branch: main
github-token: ${{ secrets.GITHUB_TOKEN }}
deploy-message: Production deployment for ${{ matrix.name }}
env:
NETLIFY_AUTH_TOKEN: ${{ secrets.NETLIFY_AUTH_TOKEN }}
NETLIFY_SITE_ID: ${{ matrix.netlifySiteId }}
7 changes: 5 additions & 2 deletions .github/workflows/staging-migrations.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Deploy staging
name: Deploy to staging

on:
pull_request:
Expand Down Expand Up @@ -45,6 +45,7 @@ jobs:
run: npm run migrations:apply-staging

prepare:
needs: apply-staging-migrations
runs-on: ubuntu-latest
outputs:
matrix: ${{ steps.set-matrix.outputs.matrix }}
Expand Down Expand Up @@ -106,7 +107,9 @@ jobs:
publish-dir: "./dist"
production-branch: main
github-token: ${{ secrets.GITHUB_TOKEN }}
deploy-message: Preview deployment for ${{ matrix.name }}
enable-commit-comment: false
enable-pull-request-comment: true
overwrites-pull-request-comment: true
env:
NETLIFY_AUTH_TOKEN: ${{ secrets.NETLIFY_AUTH_TOKEN }}
NETLIFY_SITE_ID: ${{ matrix.netlifySiteId }}

3 comments on commit 7cd62de

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Published on https://nl2120.netlify.app as production
🚀 Deployed on https://66fff7de6d7c813f6a45a508--nl2120.netlify.app

Please sign in to comment.