-
Notifications
You must be signed in to change notification settings - Fork 120
Conversation
Codecov Report
@@ Coverage Diff @@
## master #389 +/- ##
=========================================
- Coverage 95.92% 93.9% -2.02%
=========================================
Files 6 6
Lines 3309 3399 +90
Branches 76 123 +47
=========================================
+ Hits 3174 3192 +18
- Misses 134 205 +71
- Partials 1 2 +1
Continue to review full report at Codecov.
|
@@ -14,54 +14,34 @@ | |||
* limitations under the License. | |||
*/ | |||
|
|||
/** |
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.
So that's a problem, no? We have to expose the jsdoc for the still relevant v2 version of the API.
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.
version=version) | ||
s.copy(library, excludes=['README.md', 'package.json', 'src/index.ts']) |
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.
This is awesome
This PR converts the last part of the library, the
v3beta1
API, to TypeScript. Also, making sure the common resources (such aslocation
) are correctly processed for bothv3
andv3beta1
.