Merge pull request #1180 from openmina/dw/ci-tx-fuzzing-check

CI/lint: move tx fuzzer lint to lint.yaml

Author: dannywillems

.github/workflows/ci.yaml

@@ -100,25 +100,6 @@ jobs:
           cd vrf
           cargo test --release -- -Z unstable-options --report-time
 
-  tx-fuzzer-check:
-    runs-on: ubuntu-22.04
-    steps:
-      - name: Git checkout
-        uses: actions/checkout@v4
-      - name: Setup build dependencies
-        run: |
-          sudo apt update
-          sudo apt install -y protobuf-compiler
-      - name: Setup Rust
-        run: |
-          rustup install nightly
-          rustup override set nightly
-          rustup component add --toolchain nightly-x86_64-unknown-linux-gnu rustfmt
-      - name: Check for compilation errors in transaction fuzzer
-        run: |
-          cd tools/fuzzing
-          cargo check
-
   p2p-tests:
     runs-on: ubuntu-22.04
     steps:

.github/workflows/lint.yml

@@ -7,19 +7,43 @@ on:
 
 jobs:
   lint:
-    name: Lint
-    runs-on: ubuntu-latest
+    name: Lint - ${{ matrix.os }} - Rust ${{ matrix.toolchain }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        toolchain: [1.84]
     steps:
       - uses: actions/checkout@v4
       - name: Setup build dependencies
         run: |
-          sudo apt update
-          sudo apt install -y protobuf-compiler
+          sudo apt update || true
+          sudo apt install -y protobuf-compiler || true
       - uses: dtolnay/rust-toolchain@stable
         with:
-          toolchain: 1.84
+          toolchain: ${{ matrix.toolchain }}
           components: clippy, rustfmt
-      - name: check
+      - name: Run make check
         run: make check
-      - name: clippy
+      - name: Run make lint
         run: make lint
+
+  lint-tx-fuzzing:
+    name: Lint transaction Fuzzing - ${{ matrix.os }} - Rust ${{ matrix.toolchain }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        toolchain: [nightly]
+    steps:
+      - uses: actions/checkout@v4
+      - name: Setup build dependencies
+        run: |
+          sudo apt update || true
+          sudo apt install -y protobuf-compiler || true
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          toolchain: ${{ matrix.toolchain }}
+          components: clippy, rustfmt
+      - name: Run transaction Fuzzing check
+        run: make check-tx-fuzzing

Makefile

@@ -24,6 +24,10 @@ build-wasm: ## Build WebAssembly node
 check: ## Check code for compilation errors
 	cargo check --all-targets
 
+.PHONY: check-tx-fuzzing
+check-tx-fuzzing: ## Check the transaction fuzzing tools, requires nightly Rust
+	@cd tools/fuzzing && cargo +nightly check
+
 .PHONY: check-format
 check-format: ## Check code formatting
 	cargo +nightly fmt -- --check