[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you written unit tests?
[ ] Have you written unit tests that cover the negative cases (i.e.: if bad data is submitted, does the library respond properly)?
[x] This PR is associated with an existing issue?
Closing issues
Fixes parts of #1165 ("Strict tsconfig in all packages" checkbox)
Own comments
This adds strict: true to all tsconfig.json files, and removes redundant options that strict enables.
All resulting TS errors have been fixed, but some fixes might still hide already existing issues.
Note: I kept noImplicitAny: false for tests, but I can remove it if we want full strictness for test code as well.
All Submissions:
Closing issues Fixes parts of #1165 ("Strict tsconfig in all packages" checkbox)
Own comments
This adds
strict: true
to alltsconfig.json
files, and removes redundant options thatstrict
enables. All resulting TS errors have been fixed, but some fixes might still hide already existing issues. Note: I keptnoImplicitAny: false
for tests, but I can remove it if we want full strictness for test code as well.