-
Notifications
You must be signed in to change notification settings - Fork 51
build!: update to latest gts and TypeScript #851
build!: update to latest gts and TypeScript #851
Conversation
6836c05
to
c2e0eec
Compare
c2e0eec
to
0da1ea5
Compare
.eslintignore
Outdated
@@ -3,3 +3,6 @@ src/**/doc/* | |||
build/ | |||
docs/ | |||
protos/ | |||
test/fixtures/fib.js |
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.
Instead of calling these out individually, add test/fixtures
and ignore the whole dir
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.
Added
test/fixtures
system-test/fixtures
Thanks
package.json
Outdated
@@ -75,7 +75,7 @@ | |||
"cpy-cli": "^3.1.0", | |||
"cross-env": "^7.0.0", | |||
"execa": "^4.0.0", | |||
"gts": "^1.0.0", | |||
"gts": "^2.0.0-alpha.8", |
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.
Please use ^2.0.0
:)
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.
Fixed here and in system-test
Thanks
samples/test/test.js
Outdated
@@ -14,17 +14,19 @@ | |||
|
|||
'use strict'; | |||
|
|||
// eslint-disable-next-line node/no-unpublished-require |
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 looks like a bug in samples/package.json
. Can you make sure the files
property is set in that file? We shouldn't need this exclusion rule.
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.
added
"files": [
"*.js"
]
Thanks
src/agent/state/legacy-state.ts
Outdated
@@ -40,9 +40,11 @@ export function evaluate( | |||
frame: v8.FrameMirror | |||
): {error: string | null; mirror?: v8.ValueMirror} { | |||
// First validate the expression to make sure it doesn't mutate state | |||
// eslint-disable-next-line @typescript-eslint/no-var-requires |
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.
Instead of this, could you install @types/acorn
and use that?
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.
used import * as acorn from 'acorn';
Thanks
@@ -14,8 +14,8 @@ | |||
"@google-cloud/debug-agent": "./debug.tgz" | |||
}, | |||
"devDependencies": { | |||
"gts": "^1.1.0", | |||
"typescript": "~3.7.0", | |||
"gts": "^2.0.0-alpha.8", |
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.
^2.0.0
pls
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.
fixed
Thanks
var i = 1; | ||
var foo = function foo(j) { | ||
return i + j + "hi"; | ||
'use strict'; |
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.
If you get test failures, first place I'd look is rolling back changes to fixtures (otherwise this is fine)
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.
Thanks, I was looking out for the same thing. Looks like so far everything is green.
🤖 I have created a release \*beep\* \*boop\* --- ## [5.0.0](https://www.github.com/googleapis/cloud-debug-nodejs/compare/v4.2.2...v5.0.0) (2020-05-20) ### ⚠ BREAKING CHANGES * drop support for node.js 8.x (#851) * require node 10 in engines field (#852) ### Features * require node 10 in engines field ([#852](https://www.github.com/googleapis/cloud-debug-nodejs/issues/852)) ([01dc0e2](https://www.github.com/googleapis/cloud-debug-nodejs/commit/01dc0e263a57914e7f7320feac950dfedeeb7099)) ### Bug Fixes * **deps:** update dependency @google-cloud/common to v3 ([#853](https://www.github.com/googleapis/cloud-debug-nodejs/issues/853)) ([79bbc9c](https://www.github.com/googleapis/cloud-debug-nodejs/commit/79bbc9c9ad9c763166e6a96ba254c8521d485f12)) * **deps:** update dependency gcp-metadata to v4 ([#844](https://www.github.com/googleapis/cloud-debug-nodejs/issues/844)) ([8b4040e](https://www.github.com/googleapis/cloud-debug-nodejs/commit/8b4040e933676535fb75c135be931c70fc8bad37)) * apache license URL ([#468](https://www.github.com/googleapis/cloud-debug-nodejs/issues/468)) ([#863](https://www.github.com/googleapis/cloud-debug-nodejs/issues/863)) ([87553b2](https://www.github.com/googleapis/cloud-debug-nodejs/commit/87553b267df9e0008584645a80c41ef820c11689)) * reduce set of dependencies ([#876](https://www.github.com/googleapis/cloud-debug-nodejs/issues/876)) ([6c2722e](https://www.github.com/googleapis/cloud-debug-nodejs/commit/6c2722e59bb7113cf7a4bf065434b2be3aff50ca)) ### Build System * drop support for node.js 8.x ([#851](https://www.github.com/googleapis/cloud-debug-nodejs/issues/851)) ([3130ad3](https://www.github.com/googleapis/cloud-debug-nodejs/commit/3130ad320dc69b01d43d8e39d31ce158a13311f1)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
BREAKING CHANGE: typescript 3.7.x introduced breaking changes in the generated code.