Merge #273

273: Some minor improvements r=ryankurte a=eldruin

- Update codeowners
- Remove unnecessary imports
- Replace `into_iter()` with `iter()` when used on reference
- Deny warnings on cargo check and cargo test on CI (but allow them on clippy since that is a fast-moving target)

Co-authored-by: Diego Barrios Romero <[email protected]>

Author: bors[bot]

.github/CODEOWNERS

@@ -1 +1 @@
-* @rust-embedded/hal @ilya-epifanov @thejpster
+* @rust-embedded/hal

.github/workflows/ci.yml

@@ -5,6 +5,9 @@ on:
 
 name: Continuous integration
 
+env:
+  RUSTFLAGS: '--deny warnings'
+
 jobs:
   ci-linux:
     runs-on: ubuntu-latest

.github/workflows/test.yml

@@ -5,6 +5,9 @@ on:
 
 name: Test Suite
 
+env:
+  RUSTFLAGS: '--deny warnings'
+
 jobs:
   ci-linux:
     runs-on: ubuntu-latest

src/adc.rs

@@ -1,7 +1,5 @@
 //! Analog-digital conversion traits
 
-use nb;
-
 /// A marker trait to identify MCU pins that can be used as inputs to an ADC channel.
 ///
 /// This marker trait denotes an object, i.e. a GPIO pin, that is ready for use as an input to the

src/capture.rs

@@ -1,7 +1,5 @@
 //! Input capture
 
-use nb;
-
 /// Input capture
 ///
 /// # Examples

src/fmt.rs

@@ -9,9 +9,8 @@ where
 {
     fn write_str(&mut self, s: &str) -> Result {
         let _ = s
-            .as_bytes()
-            .into_iter()
-            .map(|c| nb::block!(self.write(Word::from(*c))))
+            .bytes()
+            .map(|c| nb::block!(self.write(Word::from(c))))
             .last();
         Ok(())
     }

src/serial.rs

@@ -1,7 +1,5 @@
 //! Serial interface
 
-use nb;
-
 /// Read half of a serial interface
 ///
 /// Some serial interfaces support different data sizes (8 bits, 9 bits, etc.);

src/spi.rs

@@ -1,7 +1,5 @@
 //! Serial Peripheral Interface
 
-use nb;
-
 /// Full duplex (master mode)
 ///
 /// # Notes

src/timer.rs

@@ -1,7 +1,5 @@
 //! Timers
 
-use nb;
-
 /// A count down timer
 ///
 /// # Contract