probablykasper / cpc

Text calculator with support for units and conversion
https://crates.io/crates/cpc
MIT License
121 stars 14 forks source link

turn off decimal dependency default features #33

Closed jqnatividad closed 1 year ago

jqnatividad commented 1 year ago

Hi @probablykasper ,

Since json parsing is not done anyway, can we turn off decimal's default features to get rid of https://rustsec.org/advisories/RUSTSEC-2022-0004?

resolves #32

probablykasper commented 1 year ago

I didn't consider at first that it's possible for people to depend on the rustc-serialize feature, though I doubt anyone does.

The serde and ord_subset features should probably stay though

jqnatividad commented 1 year ago

Ok @probablykasper , I amended the PR per your comment.

Also applied BetterTOML formatting while I was at it.

probablykasper commented 1 year ago

Sorry, forgot about this. Thanks for the PR!

probablykasper commented 1 year ago

Released in v1.9.1