Shouldn't set `#[deny(warnings)]` outside of CI

[tbu-]

It's brittle against new compiler versions (they might introduce new warnings) and also concretely stopped crater from finding a regression in a Rust change: rust-lang/rust#125875 (comment).

[asomers]

Yes, good point. We should switch to using -Dwarnings in CI instead.