-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TSLint to ESLint Migration #313
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## main #313 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 53 53
Lines 1445 1445
Branches 186 186
=========================================
Hits 1445 1445 Continue to review full report at Codecov.
|
Unit Test Results 1 files 49 suites 15s ⏱️ Results for commit eb92f19. |
https://github.com/angular/angular-cli/blob/master/.eslintrc.json |
@alok-traceable can you please combine the rules files so that we only have one eslintrc.json? Mainly remove base.js. |
@aaron-steinfeld We can probably take angularcli rules as a base and add/remove rules on top of it. |
My guess is that angular's own isn't particularly close to what we enforce - last I looked at the angular code, it is far more javascripty than our style. |
@alok-traceable ping. Can you please take care of this on priority? Then we will have a better idea of what rules are failing and their proper fix. |
No description provided.