-
Notifications
You must be signed in to change notification settings - Fork 889
Conversation
Thanks for your interest in palantir/tslint, @alan-agius4! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request. |
f3cd994
to
727286e
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.
Thanks for sending this in @alan-agius4! The changes under src/
look good to me. Just hoping we can work around the major version change for tslib
.
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.
Tentatively 👍 from me, assuming nobody else has anything to say.
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.
do not upgrade to tslib 2.0
e6baa9d
to
e45174d
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.
sorry for not mentioning this earlier -- pinning to a beta version feels a little more stable than dev
package.json
Outdated
@@ -73,7 +73,7 @@ | |||
"tslint-config-prettier": "^1.18.0", | |||
"tslint-plugin-prettier": "^2.0.1", | |||
"tslint-test-config-non-relative": "file:test/external/tslint-test-config-non-relative", | |||
"typescript": "~3.8.2", | |||
"typescript": "~4.0.0-dev.20200722", |
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.
can we use ~4.0.0-beta
?
Closes #4941