-
Notifications
You must be signed in to change notification settings - Fork 254
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
Feat/warp support #326
Feat/warp support #326
Conversation
… into feat/warp_support
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.
Hi John, apologies for the delayed review. This looks great though! 🥳 Do you also have time to add a couple of tests? One in e2e/tests/integration
and one in cargo-shuttle/tests/integration/run
would be terrific. Some people (including CI and I) have trouble running the tests in cargo-shuttle/tests/integration/run
currently, but we can merge this anyway if we can verify the test is sound.
Sounds good, leave it with me and I'll get them added as soon as I'm back this week. |
I think this should now be ok @oddgrd . I don't think the tests fails are related to these changes. |
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.
Looks good to me! I noticed I forgot to add salvo to one of the init tests when I was reviewing your PR, so I took the opportunity to add that. Thanks! 🥳
Adding support for the Warp web server framework with example code and
init --warp
functionality.