Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Diagnostics: don't report unused-parameter for function declarations #249

Closed
novusnota opened this issue Feb 17, 2025 · 0 comments · Fixed by #260
Closed

Diagnostics: don't report unused-parameter for function declarations #249

novusnota opened this issue Feb 17, 2025 · 0 comments · Fixed by #260
Labels
diagnostics Any diagnostic messages: from the language server, client extension or other tools, like Misti
Milestone

Comments

@novusnota
Copy link
Member

For example, the value parameter in the following declaration is reported as unused:

fun ton(value: String): Int;

I suggest that unused parameter diagnostic should only apply to definitions with some body, i.e. { ... }

@novusnota novusnota added the diagnostics Any diagnostic messages: from the language server, client extension or other tools, like Misti label Feb 17, 2025
@i582 i582 closed this as completed in #260 Feb 18, 2025
@i582 i582 closed this as completed in 8358c48 Feb 18, 2025
@i582 i582 added this to the v0.2.0 milestone Feb 18, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
diagnostics Any diagnostic messages: from the language server, client extension or other tools, like Misti
Projects
None yet
2 participants