Skip to content

noUnusedParameters incorrectly triggers with @interface JSDoc in JS files #43791

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

Open
freshp86 opened this issue Apr 23, 2021 · 2 comments
Open
Labels
Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature Suggestion An idea for TypeScript

Comments

@freshp86
Copy link

Bug Report

🔎 Search Terms

noUnusedParameters @interface

🕗 Version & Regression Information

Playground link with relevant code

💻 Code

/** @interface */
class FooInterface {
  /** @param {string} id */
  doSomething(id) {}
}

🙁 Actual behavior

Throws the following error

'id' is declared but its value is never read.

🙂 Expected behavior

There should be no error thrown, as this is the correct (and only?) way to declare interfaces with JSDoc in JavaScript files (and also with Closure Compiler)

@RyanCavanaugh RyanCavanaugh added Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature Suggestion An idea for TypeScript labels Apr 23, 2021
@RyanCavanaugh
Copy link
Member

@interface is not yet a supported JS Doc tag

@ExE-Boss
Copy link
Contributor

See #41675

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature Suggestion An idea for TypeScript
Projects
None yet
Development

No branches or pull requests

3 participants