Conversation
| * // } | ||
| */ | ||
| static createRange(start, end, key) { | ||
| static createRange(start, end, key: string) { |
There was a problem hiding this comment.
looks like... Pascal. What are the types for start and end? :)
There was a problem hiding this comment.
Added some cheap well know ones first. This library is a trip man. Will come back around for completeness again.
| * ]; | ||
| */ | ||
| value(value) { | ||
| value(value): void { |
There was a problem hiding this comment.
missing type for the parameter
| @@ -917,5 +948,4 @@ promisifyAll(Bigtable, { | |||
| module.exports = Bigtable; | |||
There was a problem hiding this comment.
This is unrelated, but I don't like this :) We can change this to proper TypeScript exports.
alexander-fenster
left a comment
There was a problem hiding this comment.
I started pointing out incomplete types but I guess it's the first (well, the third) step and we'll have more PRs. Would love to see noImplicitAny enabled for this library!
|
Trying to do this bit by bit :) I find the larger the PR the less likely it is to work 😂 But yeah, noImplicitAny coming in like a 5 PR set. |
Codecov Report
@@ Coverage Diff @@
## master #646 +/- ##
==========================================
+ Coverage 96.31% 96.33% +0.02%
==========================================
Files 15 15
Lines 12818 12892 +74
Branches 802 799 -3
==========================================
+ Hits 12346 12420 +74
Misses 469 469
Partials 3 3
Continue to review full report at Codecov.
|
No description provided.