sea-orm icon indicating copy to clipboard operation
sea-orm copied to clipboard

Distribute `sea-orm-cli` by Binstall

Open erayerdin opened this issue 3 years ago • 6 comments

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.

erayerdin avatar Feb 12 '22 20:02 erayerdin

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

billy1624 avatar Feb 13 '22 08:02 billy1624

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?

tristan-zander avatar Mar 09 '22 23:03 tristan-zander

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

billy1624 avatar Mar 10 '22 03:03 billy1624

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.

tristan-zander avatar Mar 10 '22 03:03 tristan-zander

Hahaa! Thanks!! Looking forward to the PR

billy1624 avatar Mar 10 '22 03:03 billy1624

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

aadi58002 avatar Nov 14 '22 16:11 aadi58002