[CI] Add cachepot -- part2: use cache

quasar-finance · Aug 23, 2024 · 4e1c11a · 4e1c11a
1 parent 3c041c6
commit 4e1c11a
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 14 deletions.
diff --git a/.github/workflows/all_checks_osmosis.yml b/.github/workflows/all_checks_osmosis.yml
@@ -3,6 +3,10 @@ name: Test Tube (osmosis)
 on:
   workflow_call:
     inputs:
+      unittest:
+        required: false
+        default: false
+        type: boolean
       proptest:
         required: false
         default: false
@@ -26,17 +30,17 @@ jobs:
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
       - name: Cleanup runner
-        run: sudo rm -rf /usr/local/share/boost && sudo rm -rf /usr/share/dotnet && sudo rm -rf /usr/local/lib/android && sudo rm -rf /opt/ghc && sudo rm -rf /opt/hostedtoolcache/CodeQL && sudo docker image prune --all --force
+        run: sudo rm -rf /usr/local/share/boost && sudo rm -rf /usr/share/dotnet && sudo rm -rf /usr/local/lib/android && sudo rm -rf /opt/ghc && sudo rm -rf /opt/hostedtoolcache/CodeQL && sudo docker system prune --all --force
       - name: Restore dependencies
-        id: restore-cache
+        id: restore_cache
         uses: actions/cache/restore@v4
         with:
           path: |
             ~/.cache/cachepot
             ~/.cargo
             ~/go
             ~/target
-          key: ${{ runner.os }}-cache-osmosis-${{github.sha}}
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
           restore-keys: ${{ runner.os }}-cache-osmosis
       - name: Update PATH
         run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
@@ -83,5 +87,5 @@ jobs:
             ~/.cargo
             ~/go
             ~/target
-          key: ${{ runner.os }}-cache-osmosis-${{github.sha}}
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
 
diff --git a/.github/workflows/cargo_build_cache.yml b/.github/workflows/cargo_build_cache.yml
@@ -7,6 +7,9 @@ on:
         required: true
         type: string
 
+env:
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   store-build-cache:
     runs-on: ubuntu-latest
@@ -22,11 +25,11 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
           restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
       - name: Install cachepot
-        run: test -e $HOME/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
+        run: test -e ~/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
       - name: Rust check
         run:  cargo check --all-targets --features test-tube
         working-directory: smart-contracts/${{ inputs.workspace }}
@@ -39,5 +42,5 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
diff --git a/.github/workflows/rust_basic.yml b/.github/workflows/rust_basic.yml
@@ -14,6 +14,10 @@ on:
         default: true
         type: boolean
 
+env:
+  RUSTC_WRAPPER: cachepot
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   checks:
     runs-on: ubuntu-latest
@@ -23,21 +27,25 @@ jobs:
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
       - name: Restore dependencies
+        id: restore-cache
         uses: actions/cache/restore@v4
         with:
           path: |
+            ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
-          key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-$GITHUB_SHA
-          restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
+            ~/target
+          key: ${{ runner.os }}-cache-osmosis-${{ github.sha }}
+          restore-keys: ${{ runner.os }}-cache-osmosis
+      - name: Update PATH
+        run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Rust lint
-        run: cargo lint
-        working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
+        run:  cargo clippy --features test-tube --all-targets -- -D warnings
+        working-directory: smart-contracts/${{ inputs.workspace }}
       - name: Rust format check
         run: cargo fmt --all -- --check
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
-      - name: Run unit-tests
-        if: inputs.unit_tests
+      - if: inputs.unit_tests
+        name: Run unit-tests
         run: cargo unit-test
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
diff --git a/.github/workflows/test_tube.yml b/.github/workflows/test_tube.yml
@@ -24,5 +24,9 @@ jobs:
     uses: ./.github/workflows/all_checks_osmosis.yml
     if: github.event_name == 'push'
     with:
+      unittest: true
       proptest: true
       store_deps: true
+  pre-commit:
+    uses: ./.github/workflows/all_checks_osmosis.yml
+    if: github.event_name == 'pull_request'