chore: add angular 2 specific static code validation #513
Conversation
mgechev
force-pushed
the
ng2lint
branch
4 times, most recently
from
57b5fae to
5c2af5b
Compare
| return gulp.src(src) | ||
| .pipe(plugins.tslint()) | ||
| .pipe(plugins.tslint({ | ||
| rulesDirectory: NG2LINT_RULES |
There was a problem hiding this comment.
Maybe that one should be included in tsconfig instead to leverage linting straight from the IDE.
http://palantir.github.io/tslint/usage/tslint-json/
There was a problem hiding this comment.
Yes, I intended to do so but I didn't manage to make it work. Not sure if gulp-tslint recognizes this option.
There was a problem hiding this comment.
Yep. For VSCode it should actually be added to .vscode/settings.json (https://marketplace.visualstudio.com/items?itemName=eg2.tslint)
And it works :-)
There was a problem hiding this comment.
It could still be added to tslin.json so if running tslint in a shell it will work as well.
There was a problem hiding this comment.
Yep, definitely adding it to tslint.json is a good idea.
chore: add angular 2 specific static code validation
I'm working on a static code analyzer which verifies that given project follows the best practices described in the Angular 2 Style Guide.
It will be configurable so later if any other style guide is released one be able to adjust the configuration options based on his own preferences.