Distribute `sea-orm-cli` by Binstall

SeaQL / sea-orm

🐚 An async & dynamic ORM for Rust

https://www.sea-ql.org/SeaORM/

Apache License 2.0

6.94k stars 483 forks source link

Distribute `sea-orm-cli` by Binstall #520

Open erayerdin opened 2 years ago

erayerdin commented 2 years ago

cargo-binstall is a project letting prebuilt binaries to be downloaded instead of compiling the code locally like cargo install does. This would save a lot of time.

billy1624 commented 2 years ago

Hey @erayerdin, thanks for the suggestion! Yeah, this would be a huge speedup :P

tristan-zander commented 2 years ago

I'd like to take on this task, although it doesn't seem like you have a release strategy as of now (other than manually doing so). I think that some established workflow may be necessary to include sea-orm-cli in releases, since it's not there already. Would you like to establish some form of CD to include sea-orm-cli and other binaries with releases, or rather do it manually @billy1624? Maybe mirroring a release branch alongside the standard master branch?

billy1624 commented 2 years ago

Hey @tristan-zander, thank you for the interest!

As you have said, I think we can have a GitHub Actions (CD) to to include sea-orm-cli and other binaries when new release is published.

Just like what mdbook did

Releases: https://github.com/rust-lang/mdBook/releases/tag/v0.4.15.
Workflows: https://github.com/rust-lang/mdBook/blob/master/.github/workflows/deploy.yml

tristan-zander commented 2 years ago

Oh gotcha, I didn't realize Github Actions had a hook for when you create a release. That works perfectly then. I'll try to get a PR going this weekend.

billy1624 commented 2 years ago

Hahaa! Thanks!! Looking forward to the PR

aadi58002 commented 1 year ago

Can you please close this issue as it is no longer needed https://github.com/SeaQL/sea-orm/pull/595#pullrequestreview-1072026852 ?