-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 tests for Lite custom element parser #8081
Conversation
🪼 branch checks and previews
Install Gradio from this PR pip install https://gradio-builds.s3.amazonaws.com/a0b7df5eb815efe114d0dc43dcf400567208be7a/gradio-4.27.0-py3-none-any.whl Install Gradio Python Client from this PR pip install "gradio-client @ git+https://github.com/gradio-app/gradio@a0b7df5eb815efe114d0dc43dcf400567208be7a#subdirectory=client/python" |
🦄 no changes detectedThis Pull Request does not include changes to any packages.__No changes detected. __
|
} | ||
}, | ||
entrypoint: "app.py", | ||
code: undefined, |
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.
This assertion is expected to fail (and to pass after merging the current main
) but it's already passed.
Will see why.
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.
Ah, the CI test workflow checked out the test merge commit from this PR into main
, so the fixed code on the main
was tested on the CI which was expected to be successful correctly.
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! thanks @whitphx
Description
Tests for what #8067 fixed.