Loader reporting syntax errors when dealing with typescript files about eslint-loader HOT 10 CLOSED

Hi @filipekiss
Not a problem with eslint-loader, if you remove eslint-loader, same error happens, you need loader for typescript, try this:

// webpack.config.js
....
module: {
  rules: [
    { enforce: 'pre', test: /\.ts$/, exclude: /node_modules/, loader: 'eslint-loader' },
    { test: /\.tsx?$/, loader: "ts-loader" },
  ],
},
...

from eslint-loader.

Hi @ricardogobbosouza, thank you for the explanation. It indeed worked when I added the ts-loader rule. Do you have any idea why do I need it, though? This particular case I only want to use webpack to watch-lint files.

Thanks for the help, appreciated! Cheers!

from eslint-loader.

@filipekiss webpack does not handle typescript files. See https://webpack.js.org/guides/typescript/

You can use eslint-watch

from eslint-loader.

I mean, do I still need the TS Loader for the eslint-loader in this case? Shouldn't eslint-loader just pass this to eslint and eslint itself will handle the TypeScript stuff?

(I'll close the issue because this is more curiosity than an issue related to eslint-loader. Thank you for your help!)

from eslint-loader.

@filipekiss using loader yes, you will need to add ts-loader

You can try using the plugin https://github.com/webpack-contrib/eslint-webpack-plugin

from eslint-loader.

Thank you, Ricardo! :D

from eslint-loader.

@ricardogobbosouza what is status of plugin? Any ETA wen we can deprecate loader in favor plugin?

from eslint-loader.

@evilebottnawi
The plugin was born very complete, but I need to make some adjustments, especially in the option lintDirtyModulesOnly, unfortunately we still have few downloads and therefore few reports of use.
I think we can depreciate the loader as soon as webpack 5 as released

from eslint-loader.

@evilebottnawi I just released eslint-webpack-plugin v0.1.0
I think we can already recommend the plugin, but I still doubt about deprecating eslint-loader

from eslint-loader.

Edit: never mind I fixed issue! Thank you though :)

@ricardogobbosouza Hi Ricardo, I had a similar issue so I didn't want to open a new one.

When I use typescript 'as' keyword, causes eslint-loader to throw error on the 'as' - " Parsing error: Unexpected token, expected "," " even if I add ts-loader to config. If I remove eslint-loader completely, it works fine and no errors are thrown.

Example code of as usage:

const state = reactive({
      testVariable: [] as Array<string>,
})

I do not get any other issues with typescript syntax such as const newName: string = 'hello'
currently using following:

• Operating System: linux(ubuntu)
• Node Version: v12.15
• NPM Version: v6.13.4
• webpack Version: v4.41.5
• eslint-loader Version: v3.0.2

In Nuxt environment

from eslint-loader.