-
Notifications
You must be signed in to change notification settings - Fork 7
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: ✨ list, in
operator and size
function support
#20
feat: ✨ list, in
operator and size
function support
#20
Conversation
in
operator and size
function 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.
Amazing work! Thank you for contribution! <3
There are some small thing to adjust, I left todos. I can start fixing it in 10h
in
operator and size
function supportin
operator and size
function support
Note to myself: do NOT forget to update |
Unfortunately there is a couple of todos we should address. If you really need this feature released soon, I can marge and release it in this form. Otherwise I prefer to rethink the implementation to handle edge-cases like |
Don't worry, let me check how to address them. I think I will work on them this week end. |
…nds on dyn implementation so marked as todo.
Describe your changes
Add support for lists and list operations, used spects from cel-spec.
Issue ticket number/link
Checklist before requesting a review