-
Notifications
You must be signed in to change notification settings - Fork 10.3k
R2-2317 Document SSE-C Compatibility on R2 #17167
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
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.
3 files reviewed, 5 total issue(s) found.
dcpena
left a comment
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.
Two small suggestions :) Also, thanks for fixing all of the API content!
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.
3 files reviewed, 5 total issue(s) found.
Note: We resolved prior Hyperlint review comments because:
We updated our inline suggestion AI.
We do this to avoid keeping outdated or irrelevant comments around. We'll leave a new review with current comments below.
|
This content was created before Mikkel's internship ended, but the code hasn't been merged just yet. Leaving open until I get PM go-ahead. |
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.
3 files reviewed, 3 total issue(s) found.
|
Please hold until runtime update is released |
|
@helloimalastair I added you to the group that gives you merge permissions. You should be able to merge your PR whenever it's ready. 😁 |
Summary
Updated Compatibility Matrix, API Reference, and added Example.
Please merge when feature lands.
Preview
Documentation checklist