ci: minor syntax and workflow order fixes

Author: oopsbagel

.github/workflows/main.yml

@@ -1,4 +1,4 @@
-name: Check, test, build release zip, publish docs
+name: main
 
 on:
   push:
@@ -100,6 +100,8 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       contents: read
+    env:
+      NO_FIRMWARE_BIN=true
     steps:
     - uses: actions/checkout@v4
     - name: Check formatting
@@ -110,32 +112,34 @@ jobs:
         npm install
         npm run build
         popd
-        NO_FIRMWARE_BIN=true cargo check --verbose --no-default-features --features=${{ matrix.device.name }}
+        cargo check --verbose --no-default-features --features=${{ matrix.device.name }}
     - name: Run tests
       run: |
-        NO_FIRMWARE_BIN=true cargo test --verbose --no-default-features --features=${{ matrix.device.name }}
+        cargo test --verbose --no-default-features --features=${{ matrix.device.name }}
     - name: Run clippy
       run: |
-        NO_FIRMWARE_BIN=true cargo clippy --verbose --no-default-features --features=${{ matrix.device.name }}
+        cargo clippy --verbose --no-default-features --features=${{ matrix.device.name }}
 
   windows_installer_check_and_test:
     needs: files_changed
     if: needs.files_changed.outputs.installer_changed != '0'
     runs-on: windows-latest
     permissions:
       contents: read
+    env:
+      NO_FIRMWARE_BIN=true
     steps:
       - uses: actions/checkout@v4
       - name: cargo check
         shell: bash
         run: |
           cd installer
-          NO_FIRMWARE_BIN=true cargo check --verbose
+          cargo check --verbose
       - name: cargo test
         shell: bash
         run: |
           cd installer
-          NO_FIRMWARE_BIN=true cargo test --verbose --no-default-features
+          cargo test --verbose --no-default-features
 
   build_rayhunter_check:
     if: needs.files_changed.outputs.daemon_changed != '0'
@@ -242,6 +246,7 @@ jobs:
       packages: write
     needs:
       - build_rayhunter
+      - build_rootshell
       - files_changed
       - windows_installer_check_and_test
     strategy:

.github/workflows/release.yml

@@ -16,7 +16,7 @@ jobs:
       - uses: actions/checkout@v4
       - name: Ensure all Cargo.toml files have the same version defined.
         run: |
-          defined_versions=$(find lib bin installer rootshell telcom-parser -name Cargo.toml -exec grep ^version {} \; | uniq | wc -l)
+          defined_versions=$(find lib bin installer rootshell telcom-parser -name Cargo.toml -exec grep ^version {} \; | sort -u | wc -l)
           find lib bin installer rootshell telcom-parser -name Cargo.toml -exec grep ^version {} \;
           echo number of defined versions = $defined_versions
           if [ $defined_versions != "1" ]