[Rust]:Add checks on rust code generation

Signed-off-by: Antonio Gisondi <[email protected]>
astarte-platform · Oct 5, 2023 · f91f033 · f91f033
1 parent fb45a7b
commit f91f033
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 15 deletions.
diff --git a/.github/workflows/code-generation.yaml b/.github/workflows/code-generation.yaml
@@ -87,17 +87,29 @@ jobs:
         unzip protoc-24.3-linux-x86_64.zip -d $HOME/.local
         echo "$HOME/.local/bin" >> $GITHUB_PATH
     - name: Install rust toolchain
-      uses: actions-rs/toolchain@v1
+      uses: dtolnay/rust-toolchain@master
       with:
-        profile: minimal
         toolchain: 1.59
-        override: true
+        components: clippy
     - name: Remove Rust old code
       run: (! test -f ./astarte-message-hub-proto/src/astarteplatform.msghub.rs) || rm ./astarte-message-hub-proto/src/astarteplatform.msghub.rs
-      working-directory: ./rust/rust-codegen
+      working-directory: ./rust/
     - name: Generate Rust code with cargo build
       run: cargo run -- --proto-directory ../../proto --out ../astarte-message-hub-proto/src/
       working-directory: ./rust/rust-codegen
+    - name: cargo clippy
+      run: cargo clippy --all-targets --all-features -- -D warnings
+      working-directory: ./rust/astarte-message-hub-proto
+    - name: cargo doc
+      run: cargo doc --no-deps --all-features
+      env:
+        RUSTDOCFLAGS: --cfg docsrs -D warnings
+      working-directory: ./rust/astarte-message-hub-proto
+    - name: cargo check
+      run: cargo check --all-features
+      env:
+        RUSTDOCFLAGS: -D warnings
+      working-directory: ./rust/astarte-message-hub-proto  
     - name: Upload Rust code
       uses: actions/upload-artifact@v3
       with:

diff --git a/.github/workflows/rust-check-build.yaml b/.github/workflows/rust-check-build.yaml
@@ -19,8 +19,11 @@
 name: Check main.rs formatting
 
 on:
-  pull_request:
   push:
+    branches:
+      - master
+      - release-*
+  pull_request:
 
 jobs:
   fmt:
@@ -29,13 +32,7 @@ jobs:
       - name: Checkout sources
         uses: actions/checkout@v2
       - name: Install toolchain
-        uses: actions-rs/toolchain@v1
-        with:
-          profile: minimal
-          toolchain: stable
-          override: true
-      - name: Add rustfmt
-        run: rustup component add rustfmt
+        uses: dtolnay/rust-toolchain@stable
       - name: Check formatting
         run: rustfmt --check ./src/main.rs
         working-directory: ./rust/rust-codegen
diff --git a/.github/workflows/rust-publish.yaml b/.github/workflows/rust-publish.yaml
@@ -29,11 +29,9 @@ jobs:
       - name: Checkout sources
         uses: actions/checkout@v4
       - name: Install toolchain
-        uses: actions-rs/toolchain@v1
+        uses: dtolnay/rust-toolchain@master
         with:
-          profile: minimal
           toolchain: 1.59
-          override: true
       - name: Publish astarte-message-hub-proto
         run: cargo publish --token ${CRATES_TOKEN} --dry-run
         env: