Use RUSTFLAGS to deny warnings in check.rs.

Author: orium

Cargo.toml

@@ -41,9 +41,6 @@ serde = "1.0.228"
 toml = "0.9.8"
 wildmatch = "2.5.0"
 
-[features]
-fatal-warnings = []
-
 [lib]
 # Disable libtest benchmarks so we can use criterion flags (see https://bheisler.github.io/criterion.rs/book/faq.html
 # and https://github.com/rust-lang/rust/issues/47241).

src/lib.rs

@@ -13,7 +13,6 @@
 // limitations under the License.
 
 #![no_std]
-#![cfg_attr(feature = "fatal-warnings", deny(warnings))]
 // Note: If you change this remember to update `README.md`. To do so run `cargo rdme`.
 //! `wildcard` is a rust crate for wildcard matching.
 //!

tools/check.sh

@@ -20,11 +20,13 @@ assert_installed "cargo"
 
 trap on_failure ERR
 
+export RUSTFLAGS="-Dwarnings"
+
 function check_basic {
     echo 'Building:'
-    cargo build --features fatal-warnings --all-targets
+    cargo build --all-targets --all-features
 	echo 'Testing:'
-	SHORT_BENCH=1 cargo test  --all-features --all-targets --benches
+	SHORT_BENCH=1 cargo test --all-features --all-targets --benches
 	# We test property-based tests will a much higher number of testcases. We need to compile this with the release profile,
 	# otherwise this would be very slow.
 	echo 'Testing property-based tests for longer:'
@@ -33,11 +35,11 @@ function check_basic {
     # doc tests like this.
     # See https://github.com/rust-lang/cargo/issues/6669.
     echo 'Testing doc:'
-    cargo test  --features fatal-warnings --doc
+    cargo test --doc --all-features
     echo 'Checking the benchmarks:'
-    cargo bench --features fatal-warnings -- --test
+    cargo bench --all-features -- --test
     echo 'Checking documentation:'
-    cargo doc   --features fatal-warnings --no-deps
+    cargo doc --no-deps --all-features
 }
 
 function check_doc_url_links {