-
Notifications
You must be signed in to change notification settings - Fork 12k
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
Lint script in package.json is ignored #3317
Comments
The current implementation is somewhat awkward in that ng lint uses npm run but ng test uses the webpack tslint-loader. So linting via test has a fixed configuration. |
@clydin So there is no way at this moment to ignore or exclude files from tslint? Or configure anything? |
With ng lint yes. I believe ng test doesn't lint by default anyway. |
Yes, but I want to configure tslint when I run ng test. I don't want to have two different configurations, I want ng test and ng lint to use the same config |
You can run linting while testing with |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
OS?
OSX Sierra
Versions.
Repro steps.
I've tried to change the way tslint runs when we run the command
ng test
in the scripts section of the filepackage.json
but it seems to be ignored no matter what I change. At least that's what I understood hereMention any other details that might be useful.
When I run
npm run lint
orng lint
(the same goes when I run tslint) it works as expected using the overriden config of the package.json, but when I run ng test, maybe it takes another configurationThis is the scripts section in the package.json:
This is the angular-cli.json:
The text was updated successfully, but these errors were encountered: