-
Notifications
You must be signed in to change notification settings - Fork 313
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
Fix documentation rendering #965
Conversation
Size Change: +288 kB (+0.6%) Total Size: 48.4 MB
|
Yo this is absolutely Herculean, you freakin' rock |
Since the comments clearly say "don't make this class yourself," it might make sense to mark them as `@private` so they don't show up in the documentation. The doc pages for those classes can still be visited, but they don't show up in the sidebar to the left, unless you specifically navigate to one of them. (Or, if you click a link for what the return type on the `Horizon.Server.effects()` method, for example
I don't _think_ they're accessible by SDK users, so I think it makes sense to not render them in the docs?
…ocumentation-rendering
this commit also exports the `Configuration` interface
@Shaptic I think this is ready to go! Thanks for checking it out! |
…ocumentation-rendering
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.
This rocks my socks off 🎸 AMAZING work 👏
Can you make sure you run yarn _prettier
one more time? Then I'll merge
@Shaptic when I run
If I run that replacing |
I ended up just committing all those changes in a single commit, so it'll be easy enough to revert if you want to: 841f7c9 Edit: I did a bit of digging, and it looks like commit 21bd09 from #860 is when the prettier directory changed from Another Edit: I built and previewed the docs after the prettier changes, and everything looks the same as before. Still good to go, I think |
Ah yeah we def don't want that; I keep forgetting it's only for tests. I'd like to do it for ts files but not in this commit because it'll bork a BUNCH of other PRs. |
This reverts commit 841f7c9.
…ocumentation-rendering
I'm trying to get all the JSDoc rendering to work now that the SDK has had a chance to settle a bit with its inclusion of the Soroban functionality.
Refs: #920