-
Notifications
You must be signed in to change notification settings - Fork 6
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: implement SwitchClause #117
Conversation
6fc5f2e
to
e3ecded
Compare
7fa7263
to
178a749
Compare
178a749
to
01c812a
Compare
@hfudev Hi! I have updated the documentation for the switch statement. Please take a look |
07a5f03
to
5dae444
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.
LGTM! Thanks for this nice feature.
left a few comments, and could you help add some tests on combining the depends_components+
together with the switch clauses?
efda58a
to
a28b4dd
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.
Cool. kudos to the nice test cases. LGTM!
a28b4dd
to
e7899f5
Compare
e7899f5
to
98139a0
Compare
Implemented SwitchClause to enable the ability to use the following manifest format:
or
closes #105