Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Linter not Linting #806

Closed
patrickml opened this issue Feb 8, 2017 · 5 comments
Closed

Linter not Linting #806

patrickml opened this issue Feb 8, 2017 · 5 comments

Comments

@patrickml
Copy link

Issue Type

Bug

Issue Description

The other day atom completely broke down on my computer and wouldn't open. So i wiped the atom cson file which seemed to fix it. After that I noticed that the linter wasn't working. I went on for a few days not caring too much and decided to fix it when i could. Well... i've spent countless hours trying to fix it. I've wiped the entire atom folder and re-install atom only some basic plugins lint the linter and eslint.

I am still having the same issue as before. I've tried coping working cson files from other machines for the linter and eslint, but nothing has worked. What is odd is the "autofix" seems to still fix things, but no errors ever show.

eslint from the cli works just fine

Bug Checklist

  • Restart Atom
  • Verify the eslint CLI gives the proper result, while linter-eslint does not
  • Paste the output of the Linter Eslint: Debug command from the Command Palette below
Atom version: 1.13.1
linter-eslint version: 8.1.0
ESLint version: 3.5.0
Hours since last Atom restart: 0.1
Platform: darwin
Using local project ESLint from /Users/patricklewis/Documents/sites/uTu/console/node_modules/eslint
linter-eslint configuration: {
  "fixOnSave": true,
  "lintHtmlFiles": false,
  "useGlobalEslint": false,
  "showRuleIdInMessage": true,
  "disableWhenNoEslintConfig": true,
  "eslintrcPath": "",
  "globalNodePath": "",
  "advancedLocalNodeModules": "",
  "eslintRulesDir": "",
  "disableEslintIgnore": false,
  "disableFSCache": false,
  "scopes": [
    "source.js",
    "source.jsx",
    "source.js.jsx",
    "source.babel",
    "source.js-semantic"
  ],
  "rulesToSilenceWhileTyping": [],
  "rulesToDisableWhileFixing": []
}
@patrickml
Copy link
Author

I seem to be getting an error in the developer console https://cl.ly/1m2S0u1T031H

@Arcanemagus
Copy link
Member

Are you using eslint-import-resolver-webpack? Your error makes this look like a duplicate of #796.

@patrickml
Copy link
Author

Issue was caused by web pack config containing a const same issue as #796 (comment)

@nelson6e65
Copy link

After update to Atom 1.28.0 it was not working for *.js files. So, I had to add 'javascript' to the scopes config. Now is working.

@skylize
Copy link
Contributor

skylize commented Jul 3, 2018

Thanks for letting us know @nelson6e65.

But please try to avoid spamming unrelated topics. Instead, if you cannot find your problem being discussed already, just open a new Issue.

The need to add javascript to the scopes if using the new Tree Sitter parser was documented in #1076, among other places. It has already been fixed in master branch, waiting for a release.

The suggested workaround for now is to add it yourself to the list as you have done. Alternatively, you can disable Tree Sitter, and use the older grammar parser.

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

No branches or pull requests

4 participants