ci: improve ci (#37)

jokerttu · web-flow · commit c978b0f764f9 · 2024-05-17T19:54:46.000+03:00
- add concurrency rules to workflows
- pin flutter version to 3.19.x in CI
- install swiftformat only if cache was not hit
- update actions cache from v3 to v4
- ci: use ubuntu-latest instead of ubuntu-latest-8core
diff --git a/.github/workflows/analyze.yaml b/.github/workflows/analyze.yaml
@@ -19,75 +19,82 @@ on:
     branches:
       - main
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   flutter-analyze:
     timeout-minutes: 45
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
-      - name: 'Run Flutter Analyze'
+          melos-version: "3.0.1"
+      - name: "Run Flutter Analyze"
         run: melos run flutter-analyze
 
   format:
     runs-on: ubuntu-latest
     timeout-minutes: 20
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - uses: actions/setup-java@v4
         with:
-          distribution: 'temurin'
-          java-version: '17'
+          distribution: "temurin"
+          java-version: "17"
       - name: Restore Cache
         uses: actions/cache/restore@v4
         id: cache
         with:
           path: /home/linuxbrew/.linuxbrew
-          key: ${{ runner.os }}-linuxbrew-
+          key: ${{ runner.os }}-linuxbrew
       - uses: Homebrew/actions/setup-homebrew@master
-      - name: 'Install swiftformat v0.53'
+      - name: "Install swiftformat v0.53"
+        if: steps.cache.outputs.cache-hit != 'true'
         run: |
           export HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK=1
           curl -O https://raw.githubusercontent.com/Homebrew/homebrew-core/86f85aaa82beba49f8a5aabf3a22508c9249f188/Formula/s/swiftformat.rb
           grep -qxF 'brew "swiftformat.rb"' Brewfile || echo 'brew "swiftformat.rb"' >> Brewfile
           brew bundle --force
       - name: Save Cache
         uses: actions/cache/save@v4
-        if: endsWith(steps.cache.outputs.cache-matched-key,hashFiles('Brewfile.lock.json')) == false
+        if: steps.cache.outputs.cache-hit != 'true'
         with:
           path: /home/linuxbrew/.linuxbrew
-          key: ${{ runner.os }}-linuxbrew-${{ hashFiles('Brewfile.lock.json') }}
+          key: ${{ runner.os }}-linuxbrew
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
-      - name: 'Formatter version'
+          melos-version: "3.0.1"
+      - name: "Formatter version"
         run: |
           swiftformat --version
-      - name: 'Run format'
+      - name: "Run format"
         if: ${{ success() || failure() }}
         run: |
           melos run format
-      - name: 'Validate formatting'
+      - name: "Validate formatting"
         if: ${{ success() || failure() }}
         run: |
           ./.github/workflows/scripts/validate-formatting.sh
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
@@ -19,152 +19,158 @@ on:
     branches:
       - main
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   test-dart:
     timeout-minutes: 45
-    runs-on: 
-      labels: ubuntu-latest-8core
+    runs-on:
+      labels: ubuntu-latest
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
       - uses: actions/setup-java@v4
         with:
-          distribution: 'temurin'
-          java-version: '17'
-          cache: 'gradle'
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+          distribution: "temurin"
+          java-version: "17"
+          cache: "gradle"
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
-      - name: 'Run flutter test'
+          melos-version: "3.0.1"
+      - name: "Run flutter test"
         run: melos run test:dart
 
   test-android:
     timeout-minutes: 45
-    runs-on: 
-      labels: ubuntu-latest-8core
+    runs-on:
+      labels: ubuntu-latest
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
       - uses: actions/setup-java@v4
         with:
-          distribution: 'temurin'
-          java-version: '17'
-          cache: 'gradle'
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+          distribution: "temurin"
+          java-version: "17"
+          cache: "gradle"
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          flutter-version: '3.16.x'
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
-      - name: 'Run Android native unit tests'
+          melos-version: "3.0.1"
+      - name: "Run Android native unit tests"
         run: melos run test:android
 
   test-ios:
     timeout-minutes: 45
-    runs-on: 
+    runs-on:
       labels: macos-latest-xlarge
     strategy:
       matrix:
-        working_directory:
-          ['example']
+        working_directory: ["example"]
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
       - uses: actions/cache@v4
         with:
-          path: '**/Pods'
+          path: "**/Pods"
           key: ${{ runner.os }}-pods-${{ hashFiles('**/*.podspec') }}
           restore-keys: |
             ${{ runner.os }}-pods-
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          flutter-version: '3.16.x'
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
+          melos-version: "3.0.1"
       - name: Generate necessary files with flutter build
         working-directory: ${{ matrix.working_directory }}
-        run: flutter build ios --config-only         
-      - name: 'Run iOS native unit tests'
+        run: flutter build ios --config-only
+      - name: "Run iOS native unit tests"
         run: DEVICE='iPhone 14 Pro' melos run test:ios
 
   build-android:
-    needs: [test-dart,test-android,test-ios]
+    needs: [test-dart, test-android, test-ios]
     if: contains(github.base_ref, 'main')
     timeout-minutes: 45
-    runs-on: 
-      labels: ubuntu-latest-8core
+    runs-on:
+      labels: ubuntu-latest
     env:
       MAPS_API_KEY: ${{ secrets.ACTIONS_API_KEY }}
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
       - uses: actions/setup-java@v4
         with:
-          distribution: 'temurin'
-          java-version: '17'
-          cache: 'gradle'
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+          distribution: "temurin"
+          java-version: "17"
+          cache: "gradle"
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
-      - name: 'Run build for Android'
+          melos-version: "3.0.1"
+      - name: "Run build for Android"
         run: melos run flutter-build-android
 
   build-ios:
-    needs: [test-dart,test-android,test-ios]
+    needs: [test-dart, test-android, test-ios]
     if: contains(github.base_ref, 'main')
     timeout-minutes: 90
     runs-on: macos-latest-xlarge
     env:
       MAPS_API_KEY: ${{ secrets.ACTIONS_API_KEY }}
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
-      - uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+      - uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
           architecture: x64
       - uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
-          melos-version: '3.0.1'
+          melos-version: "3.0.1"
       - uses: actions/cache@v4
         with:
-          path: '**/Pods'
+          path: "**/Pods"
           key: ${{ runner.os }}-pods-${{ hashFiles('**/*.podspec') }}
           restore-keys: |
             ${{ runner.os }}-pods-
-      - name: 'Run build for iOS'
+      - name: "Run build for iOS"
         run: melos run flutter-build-ios
diff --git a/.github/workflows/licence-check.yaml b/.github/workflows/licence-check.yaml
@@ -19,34 +19,39 @@ on:
     branches:
       - main
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   check-files-license-headers:
     runs-on: ubuntu-latest
     timeout-minutes: 30
     steps:
-      - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11
+      - uses: actions/checkout@44c2b7a8a4ea60a981eaca3cf939b5f4305c123b
       - name: Override nav sdk dependencies
         uses: ./.github/actions/dependency-override
         with:
           NAV_SDK_DEPLOY_KEY: ${{ secrets.NAV_SDK_GIT_REPO_DEPLOY_KEY }}
           NAV_SDK_DEPENDENCY_OVERRIDE: ${{ secrets.NAV_SDK_OVERRIDE }}
       - uses: actions/setup-go@0c52d547c9bc32b1aa3301fd7a9cb496313a4491
         with:
-          go-version: '^1.13.1'
+          go-version: "^1.13.1"
       # Go is used by addlicense command (addlicense is used in melos run
       # check-license-header)
       - run: go install github.com/google/addlicense@latest
       - name: Install Flutter
-        uses: subosito/flutter-action@2783a3f08e1baf891508463f8c6653c258246225
+        uses: subosito/flutter-action@44ac965b96f18d999802d4b807e3256d5a3f9fa1
         with:
-          channel: 'stable'
+          flutter-version: "3.19.x"
+          channel: "stable"
           cache: true
       - name: Install Melos
         uses: bluefireteam/melos-action@720a109b686f61979b4f9f0d14f582ab1637647e
         with:
           # Running `melos bootstrap` is not needed because we use Melos just
           # for the `check-license-header` script.
           run-bootstrap: false
-          melos-version: '3.0.1'
+          melos-version: "3.0.1"
       - name: Check license header
         run: melos run check-license-header
diff --git a/.github/workflows/release-please.yml b/.github/workflows/release-please.yml
@@ -33,3 +33,6 @@ jobs:
           release-type: dart
           package-name: "@googlemaps/flutter-driver-sdk"
           bump-minor-pre-major: true
+          bump-patch-for-minor-pre-major: true
+          prerelease: true
+          prerelease-type: beta
