-
Notifications
You must be signed in to change notification settings - Fork 198
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
Added lint to packages missing it and made linting a final build step. #733
Conversation
I think this is also being done in #677 ? |
packages/workspaces/tslint.json
Outdated
@@ -1,60 +1,61 @@ | |||
{ |
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.
It looks like this has both /tslint.json and /packages/workspaces/tslint.json?
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.
Resolved: workspaces/tslint.json is now just extending /tslint.json
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.
Why have both at all?
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.
ping?
@TimvdLippe oh my -- you're right. Reviewing #677 right now. |
Merged #677 and resolved conflicts in this branch. |
Fixed: I had moved lines in JSON object but didn't maintain comma integrity!
…nalyzer build not include depcheck which is expensive.
curly: true
and fixed code where lint reported typings were too vague.