-
Notifications
You must be signed in to change notification settings - Fork 19
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 documentation for new is
operator
#67
Conversation
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.
Overall content looks good, although I'll leave the final approval to @AMZ-brandon.
Please also update docs/collections/_other/doc-history.md to reflect that the is
operator corresponds to a new language version (2.2.0?).
Cedar SDK 3.0 will also have to be language 3.0; although |
I'm putting together a preview in my test repo. This branch is behind main - can you update it from the base branch? |
fe596c8
to
475cb28
Compare
updated |
Thanks for updating! I've pulled the branch - preview here. I am updating the document history and adding syntax highlighting to What should be the new language version for the doc history? 3.0? And what PR in the Cedar repo matches this update? |
Yes. This will be released with version 3.0 (to be release soon).
|
Let me know if that doc history update looks accurate. Otherwise LGTM & will merge. |
We should wait on the merge until the 3.0 release happens so we don't have |
Got it, standing by until I hear of the release. |
Adding docs for
is
operator which will be released in the upcoming Cedar 3.0 release.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.