-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
Signed-off-by: Sun Tan <sutan@redhat.com>
if you want to enforce the rule: https://github.com/eclipse/theia-generator-plugin/blob/master/tslint.json#L3-L7 |
I guess I can reuse this tslint.json file for all our project |
package.json
Outdated
@@ -3,10 +3,14 @@ | |||
"scripts": { | |||
"prepare": "lerna run prepare", |
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.
maybe have tslint in prepare or build task ?
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.
done
package.json
Outdated
"rebuild:electron": "theia rebuild:electron" | ||
"rebuild:electron": "theia rebuild:electron", | ||
"tslint-fix": "tslint --fix --project github-extension", | ||
"tslint": "tslint --fix --project github-extension" |
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.
in your example I don't see differences between tslint and tslint-fix
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.
done: removed --fix in the tslint one
Signed-off-by: Sun Tan <sutan@redhat.com>
eclipse-che/che#10465