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
chore(docs): reduce warnings when generating docs #6138
Conversation
This is a bunch of small miscellaneous fixes that reduce the amount of warnings logged when generating our new docs. The long term goal is to get this list down to 0 warnings, but I'll do it in multiple PRs.
@@ -27,6 +27,9 @@ | |||
}, | |||
|
|||
"extractorMessageReporting": { | |||
"ae-internal-missing-underscore": { | |||
"logLevel": "none" |
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 has been disabled because the docs tool wants anything marked as @internal
to be prefixed with an underscore. We don't follow this convention (nor do I really want to) so I've disabled this.
@@ -33,7 +33,7 @@ type KeyDescription = Required< | |||
* {@link Keyboard.up}, and {@link Keyboard.sendCharacter} | |||
* to manually fire events as if they were generated from a real keyboard. | |||
* | |||
* On MacOS, keyboard shortcuts like `⌘ A` -> Select All do not work. | |||
* On MacOS, keyboard shortcuts like `⌘ A` -\> Select All do not work. |
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.
we need to escape HTML chars in TSDoc comments
/** | ||
* @public | ||
*/ | ||
export type MouseButton = 'left' | 'right' | 'middle'; |
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.
just simplified this type to make it easier to use and document
This is a bunch of small miscellaneous fixes that reduce the amount of
warnings logged when generating our new docs. The long term goal is to
get this list down to 0 warnings, but I'll do it in multiple PRs.