-
Notifications
You must be signed in to change notification settings - Fork 218
Fixes to reference doc generation #690
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
Conversation
laurenzlong
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.
This is looking good!
export type CanonicalErrorCodeName can also be hidden.
src/providers/https.ts
Outdated
| rawRequest: Request; | ||
| } | ||
|
|
||
| // The allowed interface for an http request for a callable function. |
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.
I know we're hiding this, but just to document it better in the source code, can you revise this to:
The allowed interface for an HTTP request to a Callable function.
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.
Done.
src/providers/https.ts
Outdated
| } | ||
|
|
||
| /** @hidden */ | ||
| // The format for the http body response to a callable function. |
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.
I know we're hiding this, but just to document it better in the source code, can you revise this to:
The format for an HTTP body response from a Callable function.
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.
Done.
src/utilities/assertions.ts
Outdated
| @@ -1,3 +1,4 @@ | |||
| /** @hidden */ | |||
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.
I think this can be combined with the lines below to make one comment block:
/** @hidden
* @file Provides common assertion...
....
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.
Done.
src/utilities/path.ts
Outdated
| @@ -1,3 +1,4 @@ | |||
| /** @hidden */ | |||
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.
I think this can be combined with the lines below to make one comment block
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.
Done.
Attempting a cleanup of our reference doc generation in these areas:
-- Adding home page content.
-- Hiding things to prevent unnecessary pages from being generated.
-- Hiding specific things to exclude them from generated pages.
-- Adding missing things such as a TOC entry for CallableContext.
Staged internally at: https://firebase.devsite.corp.google.com/docs/reference/functions/providers_https_
Thanks!