comprehensive-rust icon indicating copy to clipboard operation
comprehensive-rust copied to clipboard

Remove unused main() function in day-2.morning exercise.

Open gendx opened this issue 2 years ago • 3 comments

This exercise is about making the unit tests pass, so we can remove the main() function, which is empty and unused. Otherwise the Rust playground complains about dead code.

gendx avatar Feb 03 '23 18:02 gendx

Looks like the cargo build check is not happy about that.

gendx avatar Feb 06 '23 16:02 gendx

Looks like the cargo build check is not happy about that.

Yeah, it's probably listed as a binary in the Cargo.toml file and so the build breaks without a main.

mgeisler avatar Feb 09 '23 06:02 mgeisler

We could unlist it there if it's covered equivalently by mdbook test.

mgeisler avatar Feb 09 '23 06:02 mgeisler

This looks a bit stale — we can revisit it after #1073 is merged.

mgeisler avatar Nov 02 '23 17:11 mgeisler