https://bugzilla.redhat.com/show_bug.cgi?id=2183742
Aleksei Bavshin alebastr89@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Doc Type|--- |If docs needed, set a value CC| |alebastr89@gmail.com Flags| |fedora-review+ Status|NEW |POST Assignee|nobody@fedoraproject.org |alebastr89@gmail.com
--- Comment #2 from Aleksei Bavshin alebastr89@gmail.com --- Approved.
# * missing dev-dependencies: snapbox
Good thing: tests currently do not require snapbox. Bad thing: that's because `include.workspace = true` in Cargo.toml resolves to the list from https://github.com/clap-rs/clap/blob/0.2.10/Cargo.toml#L17 which does not include 'tests/**/*' ¯_(ツ)_/¯
There are no tests to run, but if there were any, they would still require snapbox. With that in mind, `%bcond_with check` seems fine to me. You might want to check with the clap upstream if the omission was intentional though.