tslint-loader icon indicating copy to clipboard operation
tslint-loader copied to clipboard

fails if you lack module.exports.tslint.configuration (if you put your tslint config in a separate file)

Open wcarmon opened this issue 8 years ago • 8 comments

this fails with tslint 3.15.x and tslint 4.0.x

// on 4.0.x Module build failed: TypeError: Cannot read property 'findConfiguration' of undefined at resolveOptions (.../node_modules/tslint-loader/index.js:28:63)

// on 3.15.x Module build failed: TypeError: Cannot read property 'findConfiguration' of undefined at resolveOptions (.../node_modules/tslint-loader/index.js:26:63)

Workaround is to have an empty configuration object in webpack.config.js (at module.exports.tslint.configuration)

wcarmon avatar Dec 07 '16 21:12 wcarmon

it doesnt seem to work for me.. i have "tslint-loader": "^3.3.0", & "tslint": "^3.15.1", I still get

TypeError: Cannot read property 'findConfiguration' of undefined

piq9117 avatar Dec 15 '16 08:12 piq9117

@piq9117, @wcarmon - were you able to solve this issue? I am using webpack2 rc3

vlio20 avatar Jan 01 '17 20:01 vlio20

By updating to versions 4.3.1 of tslint and 3.3.0 of tslint-loader the error ceases to exist.

afnpires avatar Jan 11 '17 09:01 afnpires

It needs Tslint 4.0+ and there is also a test case for no tslint configuration, does the bug still exist?

sonicoder86 avatar Feb 12 '17 15:02 sonicoder86

Added explicit check for Tslint version in code and documentation.

sonicoder86 avatar Feb 12 '17 15:02 sonicoder86

running into something similar, but only if I'm consuming typescript from within node_modules (in my case, peer scoped packages, npm linked together.)

ollwenjones avatar Mar 09 '17 18:03 ollwenjones

@wcarmon can you provide an example repository?

sonicoder86 avatar Apr 22 '17 08:04 sonicoder86

We gave up on that approach and just run linting directly to avoid any of these issues.

On Apr 22, 2017 04:38, "Gábor Soós" [email protected] wrote:

@wcarmon https://github.com/wcarmon can you provide an example repository?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/wbuchwalter/tslint-loader/issues/57#issuecomment-296358083, or mute the thread https://github.com/notifications/unsubscribe-auth/AHEEt30198kwzadSj1TR7nlm8_Jc80Ruks5rybx3gaJpZM4LHIg_ .

wcarmon avatar Apr 22 '17 14:04 wcarmon