-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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
d3-quadtree: JsDoc, strictNullChecks and TS 2.3 #27648
Conversation
trailing whitespace TypeScript Version: 2.3
@denisname Thank you for submitting this PR! 🔔 @tomwanzek @gustavderdrache @borisyankov - please review this PR in the next few days. Be sure to explicitly select If no reviewer appears after a week, a DefinitelyTyped maintainer will review the PR instead. |
Just add a |
After 5 days, no one has reviewed the PR 😞. A maintainer will be reviewing the PR in the next few days and will either merge it or request revisions. Thank you for your patience! |
* Add JsDoc and use default generics * Activate strictNullChecks and use ExpectError * Fix most linting trailing whitespace TypeScript Version: 2.3 * Fix void-return * Add length property to internal and leaf nodes
Add JsDoc.
Activate strictNullChecks: type definition where already complete, so I just had to update the tests.
For strictFunctionTypes I have nothing to add.
TS 2.4: use default generics for
quadtree
.In tests use
ExpectError
and change theisLeaf
witha.length
as it is the recommended way in the documentation.Related #23611.
Please fill in this template.