-
-
Notifications
You must be signed in to change notification settings - Fork 202
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
Glimmer Processor can't handle errors without lines #1683
Comments
hmm -- how aren't there lines? |
message: {
ruleId: null,
fatal: true,
severity: 2,
message: 'Parsing error: ESLint was configured to run on `<tsconfigRootDir>/addons/luna/addon/components/util/cache-state.gts/0_/Users/pwagenet/Development/AuditBoard/auditboard-frontend/addons/luna/addon/components/util/cache-state.gts/0_/Users/pwagenet/Development/AuditBoard/auditboard-frontend/addons/luna/addon/components/util/cache-state.gts/0_/Users/pwagenet/Development/AuditBoard/auditboard-frontend/addons/luna/addon/components/util/cache-state.gts` using `parserOptions.project`: <tsconfigRootDir>/../../../../../users/pwagenet/development/auditboard/auditboard-frontend/tsconfig.json\n' +
'However, that TSConfig does not include this file. Either:\n' +
"- Change ESLint's list of included files to not include this file\n" +
'- Change that TSConfig to include this file\n' +
'- Create a new TSConfig that includes this file and include it in your parserOptions.project\n' +
'See the TypeScript ESLint docs for more info: https://typescript-eslint.io/docs/linting/troubleshooting##i-get-errors-telling-me-eslint-was-configured-to-run--however-that-tsconfig-does-not--none-of-those-tsconfigs-include-this-file',
line: undefined,
column: undefined
}, |
ah! all my tests didn't have parsing errors. good call out |
This would be fixed by #1667 |
@wagenet did you manage to solve the underlying issue with typescript-eslint (.gts file not included in tsconfig.json)? I've tried adding |
@bwbuchanan unfortunately, I had to turn off type-aware lints in gts files. This is an issue others are aware of. I think we may have to update ESLint to be able to use Glint directly. |
Note that this change was reverted: |
eslint-plugin-ember/lib/preprocessors/glimmer.js
Line 107 in 1155615
Some errors do not have lines associated with them, so this does not work.
The text was updated successfully, but these errors were encountered: