-
Notifications
You must be signed in to change notification settings - Fork 25
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
Improved Code Examples in Docs #2943
Conversation
WalkthroughThe updates across various documentation and code samples reflect a shift in function naming from Changes
TipsChat with CodeRabbit Bot (
|
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.
Review Status
Actionable comments generated: 4
Configuration used: CodeRabbit UI
Files selected for processing (4)
- docs/content/get-started/define-control-points.md (2 hunks)
- docs/content/guides/per-user-rate-limiting.md (1 hunks)
- docs/content/sdk/dotnet/manual.md (2 hunks)
- docs/content/sdk/javascript/manual.md (1 hunks)
Additional comments: 2
docs/content/sdk/dotnet/manual.md (1)
- 28-34: The code snippet has been updated to use C++ syntax instead of C#. This change should be reflected in the keywords and the title of the documentation to avoid confusion. The title and keywords still reference C# and .NET, which could mislead users looking for C++ documentation.
docs/content/get-started/define-control-points.md (1)
- 69-81: The explanation of how to create a feature control point is clear and informative. It provides a good overview of the process and the role of the
StartFlow
andEnd
methods within the Aperture SDK. This section seems to be well-documented and should be helpful for users trying to implement feature control points.
Description of change
Checklist
Summary by CodeRabbit