Add CI workflow

.github/workflows/ci.yml

@@ -0,0 +1,22 @@
+name: ci
+
+on: [pull_request, push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: install Rust toolchain
+        run: rustup show active-toolchain || rustup toolchain install
+      - name: check
+        run: |
+          cargo check --workspace --all-features --all-targets
+      - name: test
+        run: |
+          cargo test --all-features --workspace
+      - name: lint
+        run: |
+          cargo fmt --check --all
+          cargo clippy --workspace --all-features --all-targets -- -Dwarnings
+          RUSTDOCFLAGS='-Dwarnings' cargo doc --workspace --all-features

src/pipe.rs

@@ -316,7 +316,10 @@ impl<'alloc, 'pipe, 'interrupt, Bus: UsbBus> Pipe<'alloc, 'pipe, 'interrupt, Bus
                 // `solo ls` crashes here as it uses command 0x86
                 Err(_) => {
                     info!("Received invalid command.");
-                    self.start_sending_error_on_channel(channel, AuthenticatorError::InvalidCommand);
+                    self.start_sending_error_on_channel(
+                        channel,
+                        AuthenticatorError::InvalidCommand,
+                    );
                     return;
                 }
             };
@@ -526,11 +529,7 @@ impl<'alloc, 'pipe, 'interrupt, Bus: UsbBus> Pipe<'alloc, 'pipe, 'interrupt, Bus
             }
 
             _ => {
-                if request.command == Command::Cbor {
-                    self.needs_keepalive = true;
-                } else {
-                    self.needs_keepalive = false;
-                }
+                self.needs_keepalive = request.command == Command::Cbor;
                 if self.interchange.state() == interchange::State::Responded {
                     info!("dumping stale response");
                     self.interchange.take_response();
@@ -625,7 +624,7 @@ impl<'alloc, 'pipe, 'interrupt, Bus: UsbBus> Pipe<'alloc, 'pipe, 'interrupt, Bus
                                 message.len()
                             );
                             let response = Response::from_request_and_size(request, message.len());
-                            self.buffer[..message.len()].copy_from_slice(&message);
+                            self.buffer[..message.len()].copy_from_slice(message);
                             self.start_sending(response);
                         }
                     }