Merge pull request #289 from DaniPopes/fix-features

fix: support features

Author: prestwich

.github/workflows/ci.yml

@@ -62,8 +62,7 @@ jobs:
         run: |
           cargo hack check \
           --feature-powerset --exclude-features nightly,generic_const_exprs \
-          --depth 1 \
-          --all-targets
+          --depth 1
 
   codecov:
     # See <https://doc.rust-lang.org/nightly/unstable-book/compiler-flags/source-based-code-coverage.html>

CHANGELOG.md

@@ -9,6 +9,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- Fixed some support features ([#289])
+
+[#289]: https://github.com/recmo/uint/pulls/289
+
 ## [1.10.0] - 2023-07-30
 
 ### Added

Cargo.toml

@@ -123,12 +123,12 @@ fastrlp = ["dep:fastrlp", "alloc"]
 num-bigint = ["dep:num-bigint", "alloc"]
 parity-scale-codec = ["dep:parity-scale-codec", "alloc"]
 primitive-types = ["dep:primitive-types"]
-proptest = ["dep:proptest", "alloc"]
+proptest = ["dep:proptest", "std"] # TODO: change to "alloc" on the next proptest release (>1.2.0)
 pyo3 = ["dep:pyo3", "std"]
 quickcheck = ["dep:quickcheck", "std"]
 rand = ["dep:rand"]
-rlp = ["dep:rlp"]
-serde = ["dep:serde"]
+rlp = ["dep:rlp", "alloc"]
+serde = ["dep:serde", "alloc"] # TODO: try to avoid alloc in serde impls
 valuable = ["dep:valuable"]
 zeroize = ["dep:zeroize"]
 

src/support/scale.rs

@@ -4,6 +4,7 @@
 #![cfg_attr(docsrs, doc(cfg(feature = "parity-scale-codec")))]
 
 use crate::Uint;
+use alloc::vec::Vec;
 use parity_scale_codec::{
     Compact, CompactAs, Decode, Encode, EncodeAsRef, EncodeLike, Error, HasCompact, Input,
     MaxEncodedLen, Output,

src/support/serde.rs

@@ -4,6 +4,7 @@
 #![cfg_attr(docsrs, doc(cfg(feature = "serde")))]
 
 use crate::{nbytes, Bits, Uint};
+use alloc::string::String;
 use core::{
     fmt::{Formatter, Result as FmtResult, Write},
     str,
@@ -174,6 +175,7 @@ impl<'de, const BITS: usize, const LIMBS: usize> Visitor<'de> for ByteVisitor<BI
 mod tests {
     use super::*;
     use crate::{const_for, nlimbs};
+    use alloc::vec::Vec;
     use proptest::proptest;
 
     #[test]