rust-secure-code / cargo-supply-chain

Gather author, contributor and publisher data on crates in your dependency graph.
Apache License 2.0
316 stars 19 forks source link

Migrate to bpaf v0.6 #91

Closed Shnatsel closed 2 years ago

Shnatsel commented 2 years ago

@pacak are the turbofishes on argument mandatory now, or is there a simpler way to do this that I'm not seeing?

If this is how it's intended to be used, then I prefer v0.5 API.

pacak commented 2 years ago

They are mandatory in 0.6 but I'm planning to get rid of them in the next version - new code already exists, maybe will be released in a week or two.

pacak commented 2 years ago

https://github.com/pacak/bpaf/pull/97

Trying to figure out what else I want to include to minimize number of breaking releases.

pacak commented 2 years ago

(migration to that version from 0.6 should be a no-op unless you add your own implementations of FromOsStr which most people don't really need)

Shnatsel commented 2 years ago

Alright. I'll merge this for now, but probably won't push to crates.io. I'm looking forward to dropping the String turbofishes in 0.7.