-
Notifications
You must be signed in to change notification settings - Fork 98
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
fix: improve TypeScript types #650
Conversation
const row = this.row!; | ||
row.key = newRowKey; | ||
row.data = {}; | ||
this.family = row.data[chunk.familyName.value] = {} as Family; |
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.
am I dreaming, or am I really seeing the good old a = b = {}
in the real-life JavaScript code? o_O
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.
Trying not to change the functionality here, but yeah....
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.
You know what, I'm taking this back! I just realized that it might be what they actually wanted to assign the same object to both this.family
and row.data[chunk.familyName.value]
. I'm afraid of this code :(
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.
Possibly unintended logic change related to RegExp
Codecov Report
@@ Coverage Diff @@
## master #650 +/- ##
==========================================
+ Coverage 96.35% 96.38% +0.03%
==========================================
Files 15 15
Lines 12993 13118 +125
Branches 809 810 +1
==========================================
+ Hits 12519 12644 +125
Misses 471 471
Partials 3 3
Continue to review full report at Codecov.
|
@alexander-fenster ready for another look (this time with tests passing!) |
No description provided.