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
build!: upgrade to gts@2 #2062
build!: upgrade to gts@2 #2062
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2062 +/- ##
==========================================
- Coverage 38.99% 38.91% -0.08%
==========================================
Files 6 6
Lines 1072 1074 +2
Branches 7 8 +1
==========================================
Hits 418 418
- Misses 654 656 +2
Continue to review full report at Codecov.
|
"main": "./build/src/index.js", | ||
"types": "./build/src/index.d.ts", | ||
"engines": { | ||
"node": ">=10" |
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 breaking right?
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.
I know I know, we do majors all the time anyway. But still.
.authenticate(scopes) | ||
.then(runSample) | ||
.catch(console.error); | ||
sampleClient.authenticate(scopes).then(runSample).catch(console.error); |
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.
I liked the old formatting more :) Maybe async
/await
here?
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 a heroic effort.
BREAKING CHANGE: Node 10 is now the minimum engines requirement.