Thanks for the PR! The main maintainers are out of office (or will be in the next 1-2 days) for the next 2-3 weeks so this may get delayed quite a bit.

P.S. Please revert the changes in Cargo.toml and package.json. Versioning will be handled in CI.

Thanks @FabianLars - Changes made, good to know versioning is handled by CI.
No urgency on this; it's been on my radar for a few weeks and wanted to get it done while I remember.

Package Changes Through c634107

There are 6 changes which include geolocation with patch, geolocation-js with patch, updater with minor, updater-js with minor, upload with minor, upload-js with minor

Add another change file through the GitHub UI by following this link.

Read about change files or the docs at github.com/jbolda/covector

what do you think about using https://docs.rs/http/1.3.1/http/method/struct.Method.html instead of a custom enum? It's already accessible via reqwest::Method or tauri::http::Method i think. (really asking for your opinion, not telling you what to do)

what do you think about using https://docs.rs/http/1.3.1/http/method/struct.Method.html instead of a custom enum? It's already accessible via reqwest::Method or tauri::http::Method i think. (really asking for your opinion, not telling you what to do)

@FabianLars Good question. Had a look and while we could use reqwest::Method it doesn't support the Deserialize trait. This means we'd need to deserialize the value as a string in Rust and then parse it using Method::from_str. While this would work we'd still need a corresponding enum in the Typescript API.

With this in mind it seems simpler/safer to just use a custom enum type in Rust. What do you think?