-
Notifications
You must be signed in to change notification settings - Fork 85
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
Implement Commenting on Tables #2011
Conversation
b270271
to
284f315
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #2011 +/- ##
==========================================
+ Coverage 90.01% 90.05% +0.04%
==========================================
Files 909 918 +9
Lines 32998 33125 +127
==========================================
+ Hits 29702 29830 +128
+ Misses 3296 3295 -1
☔ View full report in Codecov by Sentry. |
b1ae2d3
to
9df6356
Compare
Since we are updating the on-disk storage format, is there anything special we need to do? |
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.
Good job on your first PR!
Don't forget to squash your commits before merging.
af77306
to
597d834
Compare
597d834
to
7dde63c
Compare
7dde63c
to
5f59f21
Compare
No description provided.