Apply new cargo fmt rules & remove deprecated actions (#1509)

# Description

@swift-nav/algint-team

This PR should make no meaningful change to the codebase contained in
libsbp. All changes to the rust code simply are the result of running
`cargo fmt` & `cargo clippy --fix` with the latest version of stable
rust.

This should make it so the CI passes.

I also changed the github actions using actions-rs as it is deprecated

# API compatibility

Does this change introduce a API compatibility risk?

No

## API compatibility plan

If the above is "Yes", please detail the compatibility (or migration)
plan:

<!-- Provide a short explanation plan here -->

# JIRA Reference

N/A

#no_auto_pr

Author: pcrumley

.github/workflows/rust.yaml

@@ -25,24 +25,16 @@ jobs:
       - run: ./scripts/ci_prepare_rust.bash
         shell: bash
 
-      - uses: actions-rs/toolchain@v1
+      - name: Install rust stable latest
+        uses: dtolnay/rust-toolchain@stable
         with:
-          profile: minimal
-          toolchain: stable
-          override: true
-          components: rustfmt
+          components: rustfmt,clippy
 
-      - uses: actions-rs/cargo@v1
-        name: Format Check
-        with:
-          command: fmt
-          args: -- --check
+      - name: Format Check
+        run: cargo fmt -- --check
 
-      - uses: actions-rs/cargo@v1
-        name: Lint Check
-        with:
-          command: clippy
-          args: --all-features --all-targets -- -D warnings
+      - name: Lint Check
+        run : cargo clippy --all-features --all-targets -- -D warnings
 
   test:
     name: Tests
@@ -53,11 +45,8 @@ jobs:
       - run: ./scripts/ci_prepare_rust.bash
         shell: bash
 
-      - uses: actions-rs/toolchain@v1
-        with:
-          profile: minimal
-          toolchain: stable
-          override: true
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@stable
 
       - name: Run tests
         run: make test-rust
@@ -67,10 +56,13 @@ jobs:
     needs: [lint, test]
     strategy:
       matrix:
-        os:
-          - ubuntu-24.04
-          - macos-13
-          - windows-2022
+        include:
+          - os: ubuntu-24.04
+            target: x86_64-unknown-linux-musl
+          - os: macos-13
+            target: x86_64-apple-darwin
+          - os: windows-2022
+            target: x86_64-pc-windows-msvc
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v5
@@ -83,11 +75,10 @@ jobs:
       - run: ./scripts/ci_prepare_rust.bash
         shell: bash
 
-      - uses: actions-rs/toolchain@v1
+      - name: Install rust
+        uses: dtolnay/rust-toolchain@stable
         with:
-          profile: minimal
-          toolchain: stable
-          override: true
+          targets: ${{ matrix.target }}
 
       - name: Fetch Tags
         run: git fetch --tags --prune --force

rust/sbp/src/sbp_string.rs

@@ -101,10 +101,11 @@ impl SbpString<Vec<u8>, DoubleNullTerminated> {
         data: impl Into<Vec<u8>>,
     ) -> Result<Self, DoubleNullTerminatedError> {
         let vec = data.into();
-        if let [.., two, one] = vec.as_slice() {
-            if two == &0 && one == &0 {
-                return Ok(SbpString::new(vec));
-            }
+        if let [.., two, one] = vec.as_slice()
+            && two == &0
+            && one == &0
+        {
+            return Ok(SbpString::new(vec));
         };
         Err(DoubleNullTerminatedError)
     }