It’s nice to see the entire change’s message (both title and description) for more context into why the change was made.

make_changelist() returns a Result, so we should propagate the error rather than unwrap()ping it.

Thanks, I just applied the change for descriptions. However, I don’t know what the “standard” way of writing to Vec<u8> is in Rust, since that never returns Err:

https://doc.rust-lang.org/src/std/io/impls.rs.html#367-399

Alright, I understand. And I guess a panic would result in more information than bubbling up an error from writing to a Vec. Probably no need for that change, then.