-
Notifications
You must be signed in to change notification settings - Fork 300
Upgrade tslint to v5 #649
Comments
The conference app is already linted, so tslint shouldn't have any errors. |
Yes, but it should throw errors if I have any in my app. And I intentionally made some mistakes to trigger some errors. |
This would be nice as there are some rules which have apparently changed (e.g. arrow-parens) and so when linting is run manually, everything passes, but when ionic-app-scripts runs linting, there are issues (which should not be there). |
You can now trigger a non-zero exit code when linting in versino |
|
@danbucholtz does this mean that we have to wait until Ionic 2 upgrades to Angular 4+? Because they do not plan to provide support for TS 2.1+ in Angular 2.x as far as I can tell. |
Angular 4 is coming out in early march, as is Ionic 3, so at that time it will work with the latests deps.
Wilson Hobbs
CEO/Founder/Developer at Canal
11th Grade at The Lovett School
Personal Information
E-mail: wilsonhobbs1@gmail.com
Cell: (404) 719-3252
Website: www.wilsonhobbs.com (https://www.wilsonhobbs.com)
Twitter/Instagram/Facebook: @wbhob
Canal Information
E-mail: wilson@getcanal.com
Web: www.getcanal.com (https://www.getcanal.com/)
Twitter/Instagram/Facebook: @getcanal
…On Feb 10, 2017, 6:03 AM -0500, Roland Groza ***@***.***>, wrote:
@danbucholtz (https://github.com/danbucholtz) does this mean that we have to wait until Ionic 2 upgrades to Angular 4+? Because they do not plan to provide support for TS 2.1+ in Angular 2.x as far as I can tell.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (#649 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AEPIEmNDU5fEun3XwlZ5Z9Wqdk5oV2H2ks5rbEQGgaJpZM4Limxn).
|
I looked into it and we'll probably just wait for TS Lint 4. I tried doing it now but they have some Type Definitions in there that require TS 2.1, which we can't use yet until Angular supports it. So yes, most likely when V4 ships. Thanks, |
@danbucholtz now that Ionic is at Angular |
It is not a very high priority item. We'd take a well-tested PR. Thanks, |
May as well go for TSLint 5, while you're in there. ;) |
Closing this issue as the above PR got merged. Thanks @rolandjitsu |
Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
It would be nice to have tslint upgraded to v4. Because right now, it does not complain if I use that version, but
ionic-app-scripts lint
does nothing, just exits.What behavior are you expecting?
If I use v4 it should still work and log errors. Additionally it should exit with code
1
if there was an error.Steps to reproduce:
"tslint": "^4.3.1"
topackage.json
dev deps;ionic-app-scripts lint
.Which @ionic/app-scripts version are you using?
1.0.0
The text was updated successfully, but these errors were encountered: