-
Notifications
You must be signed in to change notification settings - Fork 335
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
.eslintrc.js does not have suggestions #124
Comments
Might be hard to achieve since we need to tell the tsserver about a d.ts eslint file under the cover. On top of this we would need to maintain such a file. |
Any movement on this? Not getting any linting based on this file. |
No movement so far. |
There is intellisense in |
Up - still needed! Filename /cc @dbaeumer Edit: |
Yup, there's code assist support for JSON, but conditionals are problematic. Here's a default vue setup, .eslintrc.js, with ENV conditionals.
|
Any update on this? |
As I was trying to explain this is something that needs to come from the TS / JS server. Let me ping someone in the TS team if there is any way to inject a d.ts for a specific file. |
@DanielRosenwasser is there any way I can teach the TypeScript server to check a certain file using a specific d.ts. file. For example by using some sort of meta comment in the file ? |
…pletion for key values like TSLint IntelliSense. See microsoft/vscode-eslint#124. There is IntelliSense for {.eslintrc.json}. Requires TS server .d.ts injection
Any more updates on this? |
@DanielRosenwasser any updates on comment #124 (comment) |
Sorry, I didn't see the original question. Today I think your best bet would be to install /**
* @type {import("eslint").Linter.Config}
*/
const config = {
// ...
}
module.exports = config; |
@DanielRosenwasser THANKS!! Will close the issue then since there is nothing ESLint needs to do here. |
I don't think this suggestion is ideal. You're saying add a JSDoc to every TypeScript file in my repo? |
Ahhh, no, just the config JavaScript? Interesting, lemme give this a try within the hour |
@cdbattags correct. |
@cdbattags Just to the |
Might I open/add a different ticket for the long term? Ideally I don't want to have to remember to add this annotation to each new repo/config. I don't mind attempting the work on this. |
IMO this should be handled by the |
@DanielRosenwasser The JSDoc import snippet doesn't seem to work. Intellisense works for |
From @isidorn on August 30, 2016 13:34
Refs #11100
Copied from original issue: microsoft/vscode#11213
The text was updated successfully, but these errors were encountered: