Skip to content
This repository was archived by the owner on Mar 25, 2021. It is now read-only.

fix: add support for TypeScript 4.0#4943

Merged
adidahiya merged 1 commit into
palantir:masterfrom
alan-agius4:ts4-support
Jul 30, 2020
Merged

fix: add support for TypeScript 4.0#4943
adidahiya merged 1 commit into
palantir:masterfrom
alan-agius4:ts4-support

Conversation

@alan-agius4

Copy link
Copy Markdown
Contributor

Closes #4941

@palantirtech

Copy link
Copy Markdown
Member

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.

@alan-agius4
alan-agius4 force-pushed the ts4-support branch 3 times, most recently from f3cd994 to 727286e Compare July 23, 2020 13:47
@alan-agius4
alan-agius4 marked this pull request as ready for review July 23, 2020 13:52

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread package.json Outdated

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Copy link
Copy Markdown
Contributor

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.

@adidahiya adidahiya left a comment

Copy link
Copy Markdown
Contributor

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

@alan-agius4
alan-agius4 force-pushed the ts4-support branch 2 times, most recently from e6baa9d to e45174d Compare July 29, 2020 08:31
@alan-agius4
alan-agius4 requested a review from adidahiya July 29, 2020 08:38

@adidahiya adidahiya left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread package.json Outdated
"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",

Copy link
Copy Markdown
Contributor

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?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

align rule (and possibly others) broken with TypeScript 4.0.0-dev

4 participants