Skip to content

Commit 17028e4

Browse files
bors[bot]dbrgn
andcommitted
Merge #17
17: Remove #[deny(warnings)] r=therealprof a=dbrgn The library currently does not currently build anymore, because upstream libraries have added new deprecation warnings. A deprecation warning should not break compilation though. There was a post on Reddit about this about a year ago, but I can't find it anymore. I think adding a CI check step would be fine, but `#[deny(warnings)]` is a bit too much. These are the warnings: ``` warning: use of deprecated item 'hal::digital::OutputPin': Deprecated because the methods cannot return errors. Users should use the traits in digital::v2. --> src/lib.rs:111:6 | 111 | impl hal::digital::OutputPin for Pin { | ^^^^^^^^^^^^^^^^^^^^^^^ | = note: #[warn(deprecated)] on by default warning: use of deprecated item 'hal::digital::InputPin': Deprecated because the methods cannot return errors. Users should use the traits in digital::v2. --> src/lib.rs:121:6 | 121 | impl hal::digital::InputPin for Pin { | ^^^^^^^^^^^^^^^^^^^^^^ warning: use of deprecated item 'hal::digital::InputPin::is_high': Deprecated because the methods cannot return errors. Users should use the traits in digital::v2. --> src/lib.rs:131:15 | 131 | !self.is_high() | ^^^^^^^ ``` Co-authored-by: Danilo Bargen <[email protected]>
2 parents 2960715 + 9313b7f commit 17028e4

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/lib.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
//! [0]: https://crates.io/keywords/embedded-hal
1212
1313
#![deny(missing_docs)]
14-
#![deny(warnings)]
1514

1615
extern crate cast;
1716
extern crate embedded_hal as hal;

0 commit comments

Comments
 (0)