documentation: update headings and descriptions for api reference#1220
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1220 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 169 169
Lines 10835 10835
Branches 1382 1382
=========================================
Hits 10835 10835 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Thank you for the PR! This is looking really good! One thing: the tree should not be fully expanded from the get-go; we only need to include the top-level headings. As well, don't need the "Braket SDK" heading either; in fact, the API reference home should just be what you see when you click Braket SDK.
|
|
I have pushed some changes based on your feedback. To remove the |
speller26
left a comment
There was a problem hiding this comment.
Looking great! Just a few comments.
Co-authored-by: Cody Wang <cody.wang@alumni.utoronto.ca>
Co-authored-by: Cody Wang <cody.wang@alumni.utoronto.ca>
Co-authored-by: Cody Wang <cody.wang@alumni.utoronto.ca>
Co-authored-by: Cody Wang <cody.wang@alumni.utoronto.ca>
Co-authored-by: Cody Wang <cody.wang@alumni.utoronto.ca>
e3d715a to
09a12b6
Compare
|
@speller26 I guess you have to trigger the GitHub actions workflows, right? |

*Issue #1186
Description of changes:
About the module descriptions, I reference the aws official docs, e.g., ahs.
There are private classes in the modules that I update their descriptions. If I do not add their paths to

apidoc_excluded_paths, empty pages would exist like thisTherefore, I have to explicitly list them there. If we tolerate the existence of these empty pages, we can unchange
doc/conf.py.Testing done:
Merge Checklist
General
Tests
-[x] I have run the existing tests. No breaking changes.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.