Skip to content
This repository was archived by the owner on Jan 26, 2019. It is now read-only.

fix TSLint #292

Closed
wants to merge 1 commit into from
Closed

fix TSLint #292

wants to merge 1 commit into from

Conversation

kuzvac
Copy link

@kuzvac kuzvac commented Mar 30, 2018

When TSLint is linting code, he also lints .js files in config folder, and generate "No valid rules have been specified" error.
I we exclude config folder from tsconfig.json file, and error is gone.

From fork-ts-checker-webpack-plugin we can get what files is will be linted. Currently it's all files under src and config folder. Thats because we see error "No valid rules have been specified".
If we exclude config folder from linting, only files under src folder will be linted. And error is gone. :)
For me it's fixed #246

When TSLint is linting code, he also lints .js files in config folder, and generate "No valid rules have been specified" error. 
I we exclude config folder from tsconfig.json file, and error is gone.
@DorianGrey
Copy link
Collaborator

A similar behavior should already be included by the most recent changes to the tslint.json:
78a4fd0

With that change, files from the config folder are explicitly excluded from linting.
This is not yet released, though.

@kuzvac
Copy link
Author

kuzvac commented Apr 3, 2018

Oh yes, i see, thank you.

@kuzvac kuzvac closed this Apr 3, 2018
@kuzvac kuzvac deleted the kuzvac-patch-tslint branch April 3, 2018 08:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to compile a project when using scripts v2.10.0 and up
2 participants