xd009642/tarpaulin

`--out` casing changed with release 0.27

sosthene-nitrokey opened this issue · 3 comments

sosthene-nitrokey commented

This may not be a bug, but it's not mentioned the release notes or in the commit message that introduced this so I'm not sure it's intentional.

The casing of the arguments of the --out parameter changed with the migration to Clap v4 in d1bd48e. This means that old scripts using --out Html will fail (expecting --out html).
This is changed because ValueEnum defaults to kebab-case.

This is not necessarily a bug, but it is an undocumented breaking change.

If this is not intentional (or even if it is), I think it would be great to support both casing for a while to ease the transition, maybe with a deprecation warning. This makes it easier for scripts to migrate to the new version progressively.

MiniaczQ commented

This is unintentional

xd009642 commented

reverted so it should work as before and releasing a new version as we speak

  • 🎉1
sosthene-nitrokey commented

Thanks for the quick fix!