Skip to content

AirBNB linter results in "Unable to resolve path to module" errors for Vue files #144

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

Closed
quentinneyraud opened this issue May 30, 2016 · 5 comments

Comments

@quentinneyraud
Copy link

When i run
vue-error
and then

cd test
npm install
npm run dev

I have the following issues
vue-error2

I think they come from f9f69e1, i made the same before this commit without errors and it's happen when i select AirBNB linter.

node 6.2.0
npm 3.8.9
Debian Jessie

@andrej-urvantsev
Copy link

I get the same. Template with airbnb profile is broken :(

@chrisvfritz
Copy link
Contributor

chrisvfritz commented May 30, 2016

This seems like a bug in airbnb/javascript, since Webpack is indeed able to resolve the path to the module. What would be the ideal solution for users of this linting config? A few ideas:

  • Specifically disable that rule in .eslintrc.js
  • Add .vue to those import statements, which may fix the problem
  • File an issue in the linting config so they can work better with Webpack

@chrisvfritz chrisvfritz changed the title Error with AirBNB linter AirBNB linter results in "Unable to resolve path to module" errors for Vue files May 31, 2016
@RXminuS
Copy link

RXminuS commented Jun 2, 2016

+1 ... Not really understand your guide @chrisvfritz? I'm assuming you mean there's an error in 'airbnb-base' which the .eslintrc.js extends?

In that case has someone opened an issue, and if so can we reference it here?

@chrisvfritz
Copy link
Contributor

I've just disabled the rule, since Webpack will already let you know if a module can't be resolved. a5008e6

JuliusChen pushed a commit to JuliusChen/Ceph-backup-ui that referenced this issue Jun 28, 2016
JuliusChen pushed a commit to JuliusChen/Ceph-backup-ui that referenced this issue Jun 28, 2016
Disable rule `import/no-unresolved`

vuejs-templates/webpack#144

See merge request !1
@karanssj4
Copy link

I created a gist which resolves this
https://gist.github.com/karanssj4/7188528ab36fb4e78c8fc385510f3136

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants