fix: correctly feature gate default features (#53)

Without this `cargo check --no-default-features` fails because of
missing `serde`.

---------

Signed-off-by: Wim Looman <[email protected]>
Co-authored-by: Yuri Astrakhan <[email protected]>

Author: Nemo157

Cargo.toml

@@ -10,8 +10,13 @@ categories = ["embedded", "no-std", "encoding", "parsing"]
 edition = "2021"
 rust-version = "1.74"
 
+[[example]]
+name = "file_parser"
+path = "examples/file_parser.rs"
+required-features = ["encodings"]
+
 [features]
-default = ["with-serde", "encodings"]
+default = ["serde", "encodings"]
 # Support decoding from all standard encodings, e.g. cp1251 commonly used for dbc
 encodings = ["dep:encoding_rs"]
 serde = ["dep:serde"]

justfile

@@ -35,6 +35,7 @@ build-diagram:
 # Quick compile without building a binary
 check:
     cargo check {{packages}} {{features}} {{targets}}
+    cargo check {{packages}} --no-default-features {{targets}}
 
 # Generate code coverage report to upload to codecov.io
 ci-coverage: env-info && \
@@ -43,7 +44,7 @@ ci-coverage: env-info && \
     mkdir -p target/llvm-cov
 
 # Run all tests as expected by CI
-ci-test: env-info test-fmt clippy test test-doc deny && assert-git-is-clean
+ci-test: env-info test-fmt check clippy test test-doc deny && assert-git-is-clean
 
 # Run minimal subset of tests to ensure compatibility with MSRV
 ci-test-msrv: env-info check test