Merge pull request #786 from shauneccles/repo/add_ci_workflow_triggers

Update CI/CD workflows to reduce action usage and prevent early sentry notifications
LedFx · Feb 23, 2024 · f644ae0 · f644ae0
2 parents 253d8f6 + 3a8fdbf
commit f644ae0
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 19 deletions.
diff --git a/.github/workflows/ci-build.yml b/.github/workflows/ci-build.yml
@@ -1,14 +1,16 @@
 name: CI Build LedFx
 
 on:
-  push:
-    paths-ignore:
-      - 'frontend/**'
   pull_request:
+    branches:
+      - main
     paths-ignore:
       - 'frontend/**'
-  release:
-    types: [created]
+      - 'docs/**'
+      - 'ledfx_frontend/**'
+  schedule:
+    - cron: '0 8 * * *'
+  workflow_dispatch:
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
   cancel-in-progress: true
@@ -235,7 +237,7 @@ jobs:
 
       - name: Check LedFx launches
         run: |
-          poetry run ledfx --ci-smoke-test -vv -c debug_config
+          poetry run ledfx --ci-smoke-test -vv -c debug_config --offline
       - name: Attach LedFx test log
         uses: actions/upload-artifact@v4
         if: always()
@@ -307,7 +309,7 @@ jobs:
 
       - name: Check LedFx launches
         run: |
-          poetry run ledfx --ci-smoke-test -vv -c debug_config
+          poetry run ledfx --ci-smoke-test -vv -c debug_config --offline
       - name: Attach LedFx test log
         uses: actions/upload-artifact@v4
         if: always()

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
@@ -1,18 +1,13 @@
 name: CodeQL Analysis
 
 on:
-  push:
-    branches:
-      - main
-    paths-ignore:
-      - '.*/**'
   pull_request:
     branches:
       - main
     paths-ignore:
       - '.*/**'
   schedule:
-    - cron: '0 8,20 * * *'
+    - cron: '0 20 * * *'
   workflow_dispatch:
 
 concurrency:
@@ -21,7 +16,7 @@ concurrency:
 
 env:
   CACHE_VERSION: 1
-  DEFAULT_PYTHON: 3.9
+  DEFAULT_PYTHON: 3.12
 
 jobs:
   analyze-python:

diff --git a/.github/workflows/frontend-ci.yml b/.github/workflows/frontend-ci.yml
@@ -6,11 +6,13 @@ on:
       - main
     paths:
       - 'frontend/**'
+      - '.github/workflows/**'
   pull_request:
     branches:
       - main
     paths:
       - 'frontend/**'
+      - '.github/workflows/**'
 jobs:
   build:
     name: Build

diff --git a/.github/workflows/test-build-binaries.yml b/.github/workflows/test-build-binaries.yml
@@ -4,6 +4,10 @@ on:
   push:
     paths-ignore:
       - 'frontend/**'
+      - 'docs/**'
+      - 'tests/**'
+      - '.github/ISSUE_TEMPLATE/**'
+      - '.vscode/**'
   workflow_dispatch:
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -277,7 +281,7 @@ jobs:
          name: LedFx-${{ steps.ledfx-version.outputs.ledfx-version }}-windows-x64
       - name: Run LedFx
         run: |
-          LedFx/LedFx.exe -vv --ci-smoke-test
+          LedFx/LedFx.exe -vv --ci-smoke-test --offline
   run-ledfx-osx:
     name: Test LedFx (OS X)
     runs-on: macos-latest
@@ -299,7 +303,7 @@ jobs:
         run: |
           xattr -cr ./LedFx.app
           chmod +x ./LedFx.app/Contents/MacOS/LedFx
-          ./LedFx.app/Contents/MacOS/LedFx -vv --ci-smoke-test
+          ./LedFx.app/Contents/MacOS/LedFx -vv --ci-smoke-test --offline
   run-ledfx-osx-m1:
     name: Test LedFx (OS X) (Apple Silicon)
     runs-on: flyci-macos-large-latest-m1
@@ -322,7 +326,7 @@ jobs:
         run: |
           xattr -cr ./LedFx.app
           chmod +x ./LedFx.app/Contents/MacOS/LedFx
-          ./LedFx.app/Contents/MacOS/LedFx -vv --ci-smoke-test
+          ./LedFx.app/Contents/MacOS/LedFx -vv --ci-smoke-test --offline
   create-release:
     name: Create GitHub Release
     if: startsWith(github.ref, 'refs/tags/')
@@ -465,7 +469,7 @@ jobs:
           pip install ledfx-*.whl
       - name: Run LedFx
         run: |
-          ledfx -vv --ci-smoke-test
+          ledfx -vv --ci-smoke-test --offline
   install-from-sdist:
     name: Test LedFx (PyPi sdist)
     runs-on: ubuntu-latest
@@ -495,7 +499,7 @@ jobs:
           pip install ledfx-*.tar.gz
       - name: Run LedFx
         run: |
-          ledfx --ci-smoke-test -vv
+          ledfx --ci-smoke-test -vv --offline
   publish:
     name: Publish LedFx to PyPi
     needs: [install-from-wheel, install-from-sdist, run-ledfx-windows, run-ledfx-osx, run-ledfx-osx-m1]