feat: add signing, tokio-1 to clippy test

feat: add signing to msrv check
fix: test_tcp_connections import order
revert: add default features on accident

.github/workflows/test.yml

@@ -23,7 +23,7 @@ jobs:
           components: clippy
       - uses: actions-rs-plus/clippy-check@v2
         with:
-          args: --all --all-targets --features format-generated-code
+          args: --all --all-targets --features format-generated-code --features signing --features tokio-1
 
   internal-tests:
     runs-on: ubuntu-latest
@@ -59,7 +59,7 @@ jobs:
         with:
           use-cross: true
           command: check
-          args: --all --all-targets
+          args: --all --all-targets --features signing
 
   build:
     needs: [formatting, linting, internal-tests, mavlink-dump, msrv]

mavlink-core/Cargo.toml

@@ -42,5 +42,4 @@ sha2 = { version = "0.10", optional = true }
 "serde" = ["dep:serde", "dep:serde_arrays"]
 "tokio-1" = ["dep:tokio"]
 "signing" = ["dep:sha2"]
-# DO NOT COMMIT 
-default = ["std", "tcp", "udp", "direct-serial", "serde", "tokio-1", "signing"]
+default = ["std", "tcp", "udp", "direct-serial", "serde"]

mavlink/tests/tcp_loopback_tests.rs

@@ -4,10 +4,10 @@ mod test_shared;
 mod test_tcp_connections {
     use std::thread;
 
-    #[cfg(feature = "signing")]
-    use mavlink::SigningConfig;
     #[cfg(feature = "signing")]
     use crate::test_shared;
+    #[cfg(feature = "signing")]
+    use mavlink::SigningConfig;
 
     /// Test whether we can send a message via TCP and receive it OK. This also test signing as a property of a MavConnection if the signing feature is enabled.
     #[test]