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

Update to typescript@2.6 #3320

Merged
merged 10 commits into from
Oct 31, 2017
Merged

Update to typescript@2.6 #3320

merged 10 commits into from
Oct 31, 2017

Conversation

ajafff
Copy link
Contributor

@ajafff ajafff commented Oct 14, 2017

PR checklist

Overview of change:

Update to ts@2.6.0
Remove type assertions. Fixes #3161
Enable --strictFunctionTypes compiler option

[bugfix] --project excludes all files of exteral dependencies
Fixes: #2827

Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

package.json Outdated
@@ -76,7 +76,7 @@
"ts-node": "^3.3.0",
"tslint": "^5.7.0",
"tslint-test-config-non-relative": "file:test/external/tslint-test-config-non-relative",
"typescript": "~2.5.1"
"typescript": "^2.6.0-dev.20171014"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not ready to be merged. We need to wait for the next official typescript release.
2.6.0-rc cannot be used because microsoft/TypeScript#19164 breaks the formatter tests

@adidahiya
Copy link
Contributor

Still in-progress? Or are we good with the latest nightly?

@ajafff
Copy link
Contributor Author

ajafff commented Oct 26, 2017

Code-wise I don't expect any changes. I'd like to wait for the stable release though. I expect it to be released this week.

@ajafff
Copy link
Contributor Author

ajafff commented Oct 31, 2017

@adidahiya now that ts 2.6.1 is out, this PR should be good to go

@adidahiya adidahiya merged commit 9632094 into palantir:master Oct 31, 2017
HyphnKnight pushed a commit to HyphnKnight/tslint that referenced this pull request Apr 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants