-
Notifications
You must be signed in to change notification settings - Fork 227
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
refactor(typescript):noImplicitAny for index test file #502
refactor(typescript):noImplicitAny for index test file #502
Conversation
04f3b1a
to
65094eb
Compare
Codecov Report
@@ Coverage Diff @@
## master #502 +/- ##
==========================================
- Coverage 94.97% 94.95% -0.03%
==========================================
Files 16 16
Lines 996 992 -4
Branches 87 87
==========================================
- Hits 946 942 -4
Misses 42 42
Partials 8 8
Continue to review full report at Codecov.
|
79889b3
to
8f27298
Compare
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.
Just a gentle reminder to add a few tests back, then we're good to go!
6d745d2
to
87abb7b
Compare
@praveenqlogic apologies, few merge conflicts here now. Can you merge and update? |
87abb7b
to
218eec8
Compare
@JustinBeckwith : Hi merge conflict get resolved. let me know if any change needed. |
…is#502) * fix(docs): reference docs should link to section of googleapis.dev with API reference * fix(docs): make anchors work in jsdoc
towards #313 (it's a good idea to open an issue first for discussion)