trigger deploy only for affected apps

.github/workflows/validate.yml

@@ -7,6 +7,28 @@ on:
     branches: [main]
 
 jobs:
+  changes:
+    name: Detect changes
+    runs-on: ubuntu-latest
+    outputs:
+      nestjs: ${{ steps.filter.outputs.nestjs }}
+      remix: ${{ steps.filter.outputs.remix }}
+    steps:
+      - name: Check out repository
+        uses: actions/checkout@v4
+
+      - name: Check for file changes
+        uses: dorny/paths-filter@v2
+        id: filter
+        with:
+          filters: |
+            nestjs:
+              - 'apps/nestjs/**'
+              - 'libs/types/**'
+            remix:
+              - 'apps/remix/**'
+              - 'apps/types/**'
+
   validate:
     runs-on: ubuntu-latest
 
@@ -39,10 +61,10 @@ jobs:
 
   deploy-nestjs:
     name: Deploy NestJS App
+    needs: [changes, validate]
+    if: needs.changes.outputs.nestjs == 'true' && github.ref == 'refs/heads/main'
+
     runs-on: ubuntu-latest
-    # needs: [typecheck, lint, test, build]
-    needs: [validate]
-    if: ${{ github.ref == 'refs/heads/main' }}
 
     steps:
       - name: ⬇️ Checkout repo
@@ -58,10 +80,10 @@ jobs:
 
   deploy-remix:
     name: Deploy Remix App
+    needs: [changes, validate]
+    if: needs.changes.outputs.remix == 'true' && github.ref == 'refs/heads/main'
+
     runs-on: ubuntu-latest
-    # needs: [typecheck, lint, test, build]
-    needs: [validate]
-    if: ${{ github.ref == 'refs/heads/main' }}
 
     steps:
       - name: ⬇️ Checkout repo

README.md

@@ -10,8 +10,16 @@
 - [x] create CI workflow (lint + build only)
 - [x] add simple tests to CI workflow
 - [x] manually deploy to Fly.io
-- [ ] create CI deployment workflow to Fly.io
-- [ ] improve CI deployment workflow to trigger only for changed apps
+- [x] create CI deployment workflow to Fly.io
+- [x] improve CI deployment workflow to trigger only for changed apps
 - [ ] create shared ui lib
 - [ ] setup Storybook in shared ui lib
 - [x] set unified path aliases for all apps and shared libs (done for `apps/`, because `libs/` probably don't need them anyway)
+
+## References
+
+- https://pnpm.io/next/filtering
+- https://github.com/remix-run/indie-stack/tree/main
+- https://github.com/sveltejs/kit/tree/master
+- https://github.com/nestjs/nest/tree/master/sample
+- https://fly.io/docs/reference/configuration/#the-processes-section

apps/remix/app/routes/_index.tsx

@@ -29,7 +29,7 @@ export default function Index() {
       <hr />
       <h2>Cats fetched from NestJS</h2>
       <pre>{JSON.stringify(cats, null, 2)}</pre>
-      <p>Test</p>
+      <p>Test 2</p>
     </div>
   )
 }