-
Notifications
You must be signed in to change notification settings - Fork 26
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
Validation for lightweight components #793
Conversation
One thing I noticed while writing tests:
but
does not Notice the () on the decorator. It might be nice to support both or at least throw a clear error. |
Added the support for both variants. When we call the decorator without parentheses the default values of the |
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.
Nice ! We can still expand once we discover more cases.
Adding validation to the PythonComponent.
Fix #783