search

riasvdv / laravel-mix-eslint

mix.eslint()
20 stars 5 forks source link

Vue Support #1

Closed christian-thomas closed 5 years ago

christian-thomas commented 6 years ago

At the moment the linter hasn't got support for Vue files.

It's fairly easy to add with:

test: /\.(js|vue)$/,

But will also require eslint-plugin-vue, and extending the ruleset in your eslint:

"extends": [
    "plugin:vue/recommended"
],

Is this something you'd look at adding if I update the Readme and PR? If you want me to make the Vue Support optional, I'm happy to add notes to this effect, and make the webpack rule/test customisable under the existing options config.

riasvdv commented 6 years ago

I'll gladly merge a PR for this! I don't think it has to be optional as long as it still works when you're not using Vue.

christian-thomas commented 6 years ago

No problem, I'll get a PR in asap. 👍