-
Notifications
You must be signed in to change notification settings - Fork 5.5k
docs(native): Add documentation for REST-based remote functions #26513
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
base: master
Are you sure you want to change the base?
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR extends the Presto documentation by adding comprehensive coverage for REST-based remote functions, including an overview, detailed syntax and configuration parameters, and practical usage examples—all integrated into the features.rst guide. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
@sourcery-ai review |
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.
|
Thanks for the documentation! I'll wait to review this until it's out of Draft mode. |
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.
Thanks for the documentation! This is really well-done: good organization, useful examples, and clear instructions on how to configure. A few minor nits of formatting and phrasing but overall very good. Thanks!
4f3b17e to
bee054a
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! (docs)
Pull updated branch, new local doc build. Thanks for the update!
Description
Add documentation for REST-based remote functions
Impact
NA
Test Plan
NA
Contributor checklist
Release Notes