-
Notifications
You must be signed in to change notification settings - Fork 50
Adding initial developer documentation #10
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
Adding initial developer documentation #10
Conversation
This is AI generated, so the next step will be a human review
|
This documentation was AI-generated, based on the original Features API docs. My next step will be to review the generated docs manually and update them accordingly. |
|
Thanks for the update! I see that #8 is already in progress. I’ll close my PR to avoid duplication. Happy to help with other issues in the future! |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## trunk #10 +/- ##
========================================
Coverage ? 92.48%
Complexity ? 99
========================================
Files ? 7
Lines ? 519
Branches ? 0
========================================
Hits ? 480
Misses ? 39
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I updated the branch with GitHub UI to the latest changes from |
jonathanbossenger
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.
Initial manual review complete. I just want to test all the code examples.
|
Updated all the factual errors. Last step is to test and verify the code examples, but I'm marking this as ready to review so long. |
|
|
Would it make sense to land the initial version and tackle the subsequent iterations in follow-up targeted PRs? I approved PR, so with all checks passing, you should be able to merge at any point. |
Yup, that was my plan as well. Going to merge here in a bit. |
Adds developer documentation for the Abilities API. This is still a work in progress.
Fixes #5