[INFO] cloning repository https://github.com/cgcardona/codec
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/cgcardona/codec" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcgcardona%2Fcodec", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcgcardona%2Fcodec'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 37e9a841e8a48fe291a08a5428e3198ab52558c1
[INFO] checking cgcardona/codec against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fcgcardona%2Fcodec" "/workspace/builds/worker-4-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/cgcardona/codec
[INFO] finished tweaking git repo https://github.com/cgcardona/codec
[INFO] tweaked toml for git repo https://github.com/cgcardona/codec written to /workspace/builds/worker-4-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/cgcardona/codec on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/cgcardona/codec already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762
[INFO] running `Command { std: "docker" "start" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling serde v1.0.102
[INFO] [stderr]    Compiling byteorder v1.3.2
[INFO] [stderr]     Checking hex v0.3.2
[INFO] [stderr]     Checking bitcoin_hashes v0.7.1
[INFO] [stderr]     Checking codec v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:130:24
[INFO] [stdout]     |
[INFO] [stdout] 130 |             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 130 -             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout] 130 +             return Err(Base58Error::InvalidLength(length));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 130 -             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout] 130 +             return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::InvalidLength(length)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:140:29
[INFO] [stdout]     |
[INFO] [stdout] 140 |             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 140 -             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout] 140 +             _ => return Err(Base58Error::InvalidVersion(version_byte)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 140 -             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout] 140 +             _ => return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::InvalidVersion(version_byte))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:148:24
[INFO] [stdout]     |
[INFO] [stdout] 148 |               return Err(Base58Error::ChecksumFailed {
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 149 | |                 expected: checksum_expected.to_vec(),
[INFO] [stdout] 150 | |                 actual: checksum_actual.to_vec(),
[INFO] [stdout] 151 | |             }
[INFO] [stdout] 152 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 151 -             }
[INFO] [stdout] 152 -             .into());
[INFO] [stdout] 151 +             });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 148 ~             return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::ChecksumFailed {
[INFO] [stdout] 149 |                 expected: checksum_expected.to_vec(),
[INFO] [stdout] 150 |                 actual: checksum_actual.to_vec(),
[INFO] [stdout] 151 ~             }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:136:29
[INFO] [stdout]     |
[INFO] [stdout] 136 |             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(CashAddrError::InvalidLength(length)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(length))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:178:24
[INFO] [stdout]     |
[INFO] [stdout] 178 |             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(CashAddrError::NoPrefix);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::NoPrefix));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:188:29
[INFO] [stdout]     |
[INFO] [stdout] 188 |             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string())),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidPrefix(prefix.to_string()))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:196:32
[INFO] [stdout]     |
[INFO] [stdout] 196 |                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(CashAddrError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:199:28
[INFO] [stdout]     |
[INFO] [stdout] 199 |                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(CashAddrError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:202:24
[INFO] [stdout]     |
[INFO] [stdout] 202 |             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(CashAddrError::InvalidLength(0));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(0)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:222:24
[INFO] [stdout]     |
[INFO] [stdout] 222 |             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(CashAddrError::ChecksumFailed(checksum));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::ChecksumFailed(checksum)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:245:24
[INFO] [stdout]     |
[INFO] [stdout] 245 |             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(CashAddrError::InvalidLength(body_len));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(body_len)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:255:24
[INFO] [stdout]     |
[INFO] [stdout] 255 |             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(CashAddrError::InvalidVersion(version));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidVersion(version)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:136:29
[INFO] [stdout]     |
[INFO] [stdout] 136 |             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(EarthError::InvalidLength(length)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(length))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:178:24
[INFO] [stdout]     |
[INFO] [stdout] 178 |             return Err(EarthError::NoPrefix.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(EarthError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(EarthError::NoPrefix);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(EarthError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::NoPrefix));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:188:29
[INFO] [stdout]     |
[INFO] [stdout] 188 |             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(EarthError::InvalidPrefix(prefix.to_string())),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidPrefix(prefix.to_string()))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:196:32
[INFO] [stdout]     |
[INFO] [stdout] 196 |                     return Err(EarthError::MixedCase.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(EarthError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(EarthError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(EarthError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:199:28
[INFO] [stdout]     |
[INFO] [stdout] 199 |                 return Err(EarthError::MixedCase.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(EarthError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(EarthError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(EarthError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:202:24
[INFO] [stdout]     |
[INFO] [stdout] 202 |             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(EarthError::InvalidLength(0));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(0)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:222:24
[INFO] [stdout]     |
[INFO] [stdout] 222 |             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(EarthError::ChecksumFailed(checksum));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::ChecksumFailed(checksum)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:245:24
[INFO] [stdout]     |
[INFO] [stdout] 245 |             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(EarthError::InvalidLength(body_len));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(body_len)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:255:24
[INFO] [stdout]     |
[INFO] [stdout] 255 |             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(EarthError::InvalidVersion(version));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidVersion(version)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:130:24
[INFO] [stdout]     |
[INFO] [stdout] 130 |             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 130 -             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout] 130 +             return Err(Base58Error::InvalidLength(length));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 130 -             return Err(Base58Error::InvalidLength(length).into());
[INFO] [stdout] 130 +             return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::InvalidLength(length)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:140:29
[INFO] [stdout]     |
[INFO] [stdout] 140 |             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 140 -             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout] 140 +             _ => return Err(Base58Error::InvalidVersion(version_byte)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 140 -             _ => return Err(Base58Error::InvalidVersion(version_byte).into()),
[INFO] [stdout] 140 +             _ => return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::InvalidVersion(version_byte))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `base58_errors::Base58Error`
[INFO] [stdout]    --> src/base58.rs:148:24
[INFO] [stdout]     |
[INFO] [stdout] 148 |               return Err(Base58Error::ChecksumFailed {
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 149 | |                 expected: checksum_expected.to_vec(),
[INFO] [stdout] 150 | |                 actual: checksum_actual.to_vec(),
[INFO] [stdout] 151 | |             }
[INFO] [stdout] 152 | |             .into());
[INFO] [stdout]     | |___________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 151 -             }
[INFO] [stdout] 152 -             .into());
[INFO] [stdout] 151 +             });
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 148 ~             return Err(<base58_errors::Base58Error as Into<_>>::into(Base58Error::ChecksumFailed {
[INFO] [stdout] 149 |                 expected: checksum_expected.to_vec(),
[INFO] [stdout] 150 |                 actual: checksum_actual.to_vec(),
[INFO] [stdout] 151 ~             }));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:136:29
[INFO] [stdout]     |
[INFO] [stdout] 136 |             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(CashAddrError::InvalidLength(length)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(CashAddrError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(length))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:178:24
[INFO] [stdout]     |
[INFO] [stdout] 178 |             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(CashAddrError::NoPrefix);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(CashAddrError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::NoPrefix));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:188:29
[INFO] [stdout]     |
[INFO] [stdout] 188 |             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string())),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(CashAddrError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidPrefix(prefix.to_string()))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:196:32
[INFO] [stdout]     |
[INFO] [stdout] 196 |                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(CashAddrError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:199:28
[INFO] [stdout]     |
[INFO] [stdout] 199 |                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(CashAddrError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(CashAddrError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:202:24
[INFO] [stdout]     |
[INFO] [stdout] 202 |             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(CashAddrError::InvalidLength(0));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(CashAddrError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(0)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:222:24
[INFO] [stdout]     |
[INFO] [stdout] 222 |             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(CashAddrError::ChecksumFailed(checksum));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(CashAddrError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::ChecksumFailed(checksum)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:245:24
[INFO] [stdout]     |
[INFO] [stdout] 245 |             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(CashAddrError::InvalidLength(body_len));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(CashAddrError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidLength(body_len)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cashaddr_errors::CashAddrError`
[INFO] [stdout]    --> src/cashaddr.rs:255:24
[INFO] [stdout]     |
[INFO] [stdout] 255 |             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(CashAddrError::InvalidVersion(version));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(CashAddrError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(<cashaddr_errors::CashAddrError as Into<_>>::into(CashAddrError::InvalidVersion(version)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:136:29
[INFO] [stdout]     |
[INFO] [stdout] 136 |             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(EarthError::InvalidLength(length)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             _ => return Err(EarthError::InvalidLength(length).into()),
[INFO] [stdout] 136 +             _ => return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(length))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:178:24
[INFO] [stdout]     |
[INFO] [stdout] 178 |             return Err(EarthError::NoPrefix.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(EarthError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(EarthError::NoPrefix);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 178 -             return Err(EarthError::NoPrefix.into());
[INFO] [stdout] 178 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::NoPrefix));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:188:29
[INFO] [stdout]     |
[INFO] [stdout] 188 |             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout]     |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(EarthError::InvalidPrefix(prefix.to_string())),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 188 -             _ => return Err(EarthError::InvalidPrefix(prefix.to_string()).into()),
[INFO] [stdout] 188 +             _ => return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidPrefix(prefix.to_string()))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:196:32
[INFO] [stdout]     |
[INFO] [stdout] 196 |                     return Err(EarthError::MixedCase.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(EarthError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(EarthError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 196 -                     return Err(EarthError::MixedCase.into());
[INFO] [stdout] 196 +                     return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:199:28
[INFO] [stdout]     |
[INFO] [stdout] 199 |                 return Err(EarthError::MixedCase.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(EarthError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(EarthError::MixedCase);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 199 -                 return Err(EarthError::MixedCase.into());
[INFO] [stdout] 199 +                 return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::MixedCase));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:202:24
[INFO] [stdout]     |
[INFO] [stdout] 202 |             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(EarthError::InvalidLength(0));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 202 -             return Err(EarthError::InvalidLength(0).into());
[INFO] [stdout] 202 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(0)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:222:24
[INFO] [stdout]     |
[INFO] [stdout] 222 |             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(EarthError::ChecksumFailed(checksum));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 222 -             return Err(EarthError::ChecksumFailed(checksum).into());
[INFO] [stdout] 222 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::ChecksumFailed(checksum)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:245:24
[INFO] [stdout]     |
[INFO] [stdout] 245 |             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(EarthError::InvalidLength(body_len));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 245 -             return Err(EarthError::InvalidLength(body_len).into());
[INFO] [stdout] 245 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidLength(body_len)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `earth_errors::EarthError`
[INFO] [stdout]    --> src/earth.rs:255:24
[INFO] [stdout]     |
[INFO] [stdout] 255 |             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(EarthError::InvalidVersion(version));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 255 -             return Err(EarthError::InvalidVersion(version).into());
[INFO] [stdout] 255 +             return Err(<earth_errors::EarthError as Into<_>>::into(EarthError::InvalidVersion(version)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `codec` (lib test) due to 21 previous errors
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] error: could not compile `codec` (lib) due to 21 previous errors
[INFO] running `Command { std: "docker" "inspect" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762", kill_on_drop: false }`
[INFO] [stdout] 4622e1731c79f96ba9daff36012fd4e9c6ad767b738d1c4ee9c5994fb356b762
