-
Notifications
You must be signed in to change notification settings - Fork 239
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 e2e tests for php, swift, oci, longevity #1215
Conversation
7356776
to
064851c
Compare
4603b36
to
a0d365f
Compare
7c45978
to
78a9cd9
Compare
Signed-off-by: Rajat Jindal <rajatjindal83@gmail.com>
78a9cd9
to
a15fcfa
Compare
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 great to me! Learning some Rust macro approaches :)
} | ||
|
||
#[tokio::test] | ||
#[ignore] // https://github.com/fermyon/spin/issues/1210 |
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.
I think it should function if the '/index.php' path is used (#1210 (comment))?
Anyways, follow-up material when ready!
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.
yes, i want to update the template for php to make it work out of the box without having to have explicit index.php
. in a follow up PR
yeah this one was fun to investigate and implement. I actually pulled it in a separate repo also. will publish a crate when I have some bandwidth. |
This PR covers following:
php
,python
andswift
templates.php
test is disabled right now due tohttp-php
template not working with latest spin #1210oci
longevity
test. Idea is we check-inwasm
files and they should continue to work with future version's ofspin
andFermyon Cloud