-
Notifications
You must be signed in to change notification settings - Fork 675
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 rust/rocket #19
Add rust/rocket #19
Conversation
If you want it might also be an option to specify the rust toolchain via the make and neph targets. It would probably rely on the user having rust installed via rustup, but since that's how most people install rust that should be a given. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the great commit!
Only downside is that Rocket currently requires nightly rust to be built
It's not good feature but not a big problem.
I'll switch my local rust environment to nightly.
README.md
Outdated
@@ -20,6 +20,7 @@ | |||
- [IRON](https://github.com/iron/iron) | |||
- [nickel.rs](https://github.com/nickel-org/nickel.rs) | |||
- node | |||
- [Rocket](https://rocket.rs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh whoops, how did that happen? o.O
Thanks for this!!:tada: |
Thanks for the merge 😊 |
This PR adds Rocket to the list of Rust frameworks.
Running the client seems to work fine. I haven't run the benchmarker though.
Only downside is that Rocket currently requires nightly rust to be built because of feature flags. Not sure if this is a deal-breaker for you, but it works just fine otherwise.