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
docs: update to latest typedoc and Typescript 4.2 #1507
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1507 +/- ##
=======================================
Coverage 68.65% 68.65%
=======================================
Files 136 136
Lines 6849 6849
Branches 802 802
=======================================
Hits 4702 4702
Misses 2141 2141
Partials 6 6
|
📙 Documentation preview is available from |
"exclude": ["**/*+(.spec|.test).ts"], | ||
"excludePrivate": true, | ||
"excludeProtected": true, | ||
"stripInternal": true, | ||
"excludeInternal": true, |
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.
Is there a semantic difference here, or simply a rename?
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.
yeah, reference to a doc or just common text in the PR description would be great to quickly understand the reasons behind changes 🙂
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.
Sorry, it's removed and picked up from tsconfig.json
but I have it here to be "sure"... See TypeStrong/typedoc#1469
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.
LGTM
No description provided.