GitHub: Changes the Rust linting workflow to allow it to be required. (…

…awsdocs#6154)
max-webster · Mar 15, 2024 · 9e1083e · 9e1083e
1 parent d900e7d
commit 9e1083e
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -1,13 +1,6 @@
 name: Rust
 on:
-  push:
-    paths:
-      - "rustv1/**"
-    branches:
-      - main
   pull_request:
-    paths:
-      - "rustv1/**"
   workflow_dispatch:
 permissions:
   contents: read
@@ -16,31 +9,46 @@ jobs:
     name: Lint Rust
     runs-on: aws-doc-sdk-examples_ubuntu-latest_16-core
     steps:
+      - name: Checkout files
+        uses: actions/checkout@v4
+        with:
+          sparse-checkout: |
+            .github
+            rustv1
+      - name: Get changed files
+        id: changed-files
+        uses: tj-actions/changed-files@v41
+        with:
+          files: "rustv1/**/*.{rs,toml}"
       - name: setup
+        if: steps.changed-files.outputs.any_changed == 'true'
         run: sudo apt-get update && sudo apt-get install -y libclang-dev
       - uses: dtolnay/rust-toolchain@stable
+        if: steps.changed-files.outputs.any_changed == 'true'
         with:
           toolchain: "1.70.0"
           components: clippy, rustfmt
-      - uses: actions/checkout@v3
       - name: Set Environment
+        if: steps.changed-files.outputs.any_changed == 'true'
         run: >
           export RUSTFLAGS="-D warnings" ;
           export APP_ENVIRONMENT="test"
       - name: Rust format
+        if: steps.changed-files.outputs.any_changed == 'true'
         run: >
           "$HOME/.cargo/bin/cargo" fmt --manifest-path rustv1/cross_service/Cargo.toml --all --check &&
           "$HOME/.cargo/bin/cargo" fmt --manifest-path rustv1/examples/Cargo.toml --all --check &&
           "$HOME/.cargo/bin/cargo" fmt --manifest-path rustv1/lambda/Cargo.toml --all --check &&
           "$HOME/.cargo/bin/cargo" fmt --manifest-path rustv1/webassembly/Cargo.toml --all --check
       - name: Rust lint
-        if: success() || failure()
+        if: steps.changed-files.outputs.any_changed == 'true' && (success() || failure())
         run: >
           "$HOME/.cargo/bin/cargo" clippy --manifest-path rustv1/cross_service/Cargo.toml --all &&
           "$HOME/.cargo/bin/cargo" clippy --manifest-path rustv1/examples/Cargo.toml --all &&
           "$HOME/.cargo/bin/cargo" clippy --manifest-path rustv1/lambda/Cargo.toml --all &&
           "$HOME/.cargo/bin/cargo" clippy --manifest-path rustv1/webassembly/Cargo.toml --all
       - name: Rust test
+        if: steps.changed-files.outputs.any_changed == 'true'
         run: >
           "$HOME/.cargo/bin/cargo" test --manifest-path rustv1/cross_service/Cargo.toml --all &&
           "$HOME/.cargo/bin/cargo" test --manifest-path rustv1/examples/Cargo.toml --all &&