The "Update process()" section mentions that we indeed need to pass the HashMap as an argument. As I was going through those changes, the change from Vec<u8> to Bytes went under my radar (needs a change in line 138), and although it's simple enough to detect, I thought it would make sense to include this short explanation.
How did I test this?
Ran the indicated tests in the README.md locally:
# in doc-test
cargo +nightly test
# in tutorial-code
cargo test --all
Do notice that the doc tests required me to run with RUSTFLAGS="--cfg tokio_unstable", I can update the README as well to reflect this change if that's OK.
Darksonn
commented
2 years ago
The "Update process()" section mentions that we indeed need to pass the
HashMapas an argument. As I was going through those changes, the change fromVec<u8>toByteswent under my radar (needs a change in line 138), and although it's simple enough to detect, I thought it would make sense to include this short explanation.How did I test this?
Ran the indicated tests in the README.md locally:
Do notice that the doc tests required me to run with
RUSTFLAGS="--cfg tokio_unstable", I can update the README as well to reflect this change if that's OK.