-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add support for Rust via cargo #742
Comments
Nice, it's working, @seivan. I'm planning to get this into the next minor release. ![]() |
Hell yeah, you actually managed to get it to work! Did you use #743 or Edit: One thing to make sure if is make sure is not to clutters the filesystem with whatever artifacts and builds that are made by using some I am not 100% sure how Runme currently builds and executes and what the lifecycle is. |
Will be included in next minor or patch release. |
Not exactly sure how, but I imagine it's:
Recognize Rust
Should add
"rust": "rs"
tohttps://github.com/stateful/runme/blob/654dda374d6968620bbcc2ce4a6ef0567c88535a/internal/command/command_file.go#L106
and
https://github.com/stateful/runme/blob/654dda374d6968620bbcc2ce4a6ef0567c88535a/internal/runner/command.go#L598
(Why aren't these shared?)
Figure out Rust script
My suggestion is to use
rust-script
liketo
https://github.com/stateful/runme/blob/654dda374d6968620bbcc2ce4a6ef0567c88535a/internal/command/command.go#L260
and
https://github.com/stateful/runme/blob/654dda374d6968620bbcc2ce4a6ef0567c88535a/internal/runner/command.go#L625
Run the Rust script
This looks like it runs a sample code, but I am not exactly sure, but whatever it is it could be adjusted for Rust as well.
https://github.com/stateful/runme/blob/654dda374d6968620bbcc2ce4a6ef0567c88535a/internal/command/command_file_test.go#L62
Now regarding actually running, or more specifically running anything, I strongly advice looking into this tool this would simplify running any code cell, and even take advantage of docker to deal with system dependencies.
But that's a different issue
The text was updated successfully, but these errors were encountered: