temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
278 stars 77 forks source link

[Feature Request] Publish client crate to crates.io #607

Open Sushisource opened 1 year ago

Sushisource commented 1 year ago

Regardless of the whether or not we're moving forward with a full Rust SDK, we should publish the client crate at some point so that's more easily consumable.

cretz commented 1 year ago

I don't think it should be published from this repo. Properly publishing IMO requires proper tagging which IMO we shouldn't make sdk-core have to do. It will be confusing to have to concern ourselves with SDK core tagged versions for directly-using users while SDK users are based on commits.

I would totally support a sdk-rust repo with a (alpha versioned) client crate that may mostly pub use stuff from this repo. Maybe it's not even a client create, but just a temporalio crate and we can feature gate things later if we have dependency concerns between workers and clients. The sdk-core repo can be a submodule from that sdk-rust repo too since I think relative project reference is safer than git reference from Cargo (having submodules stuck on a specific commit for other SDKs has been a great help of allowing core to move forward independently from SDKs).