-
Notifications
You must be signed in to change notification settings - Fork 225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch to using cargo-make #1040
Comments
Why not, I have been actually weighting upon that or justfile or just plain make. 😆 But I haven't got anywhere with that choice. Sure if you want to set things up that would be great. That test file get's anyway quite cumbersome to maintain. One thing to add is probably coverage as well at somepoint but haven't really got to it. |
Like what ever is the scripts could be migrated to the either of those options. And cargo make is one feasible option. |
Both cargo-make and just are supported by https://github.com/taiki-e/install-action/blob/main/TOOLS.md , which speeds up the CI installing of the tool. just has more stars - I havent used that yet, so I will try it first. |
Yeah, sure thing, for what I understand the justfile basically 60 % makefile and then the rest is build on top of it to enhance the quirks of old fashioned makefile to give smooth command running experience. For example the acitx-web is using justfile |
Would you be interested in me replacing test.sh with a Makefile.toml
https://github.com/sagiegurari/cargo-make
I can do all the CI stuff to make this work.
The text was updated successfully, but these errors were encountered: