Skip to content
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

Enhancement: Improve Hint System for Error Handling in Extension #187

Open
unLomTrois opened this issue Oct 3, 2024 · 1 comment
Open

Comments

@unLomTrois
Copy link

Hey, thank you for this cool extension, I really like it. But I find it too intrusive to see these hints all around even if I cover all my code with error handlers. So, my suggestions:

Show hints to wrap function calls in try-catch blocks or adding .catch for async functions. When the user made it, remove these diagnostics.

Inside functions, if something throws and the function doesn't have @throws field in its jsdoc comment, show a warning on the function's name. When the user adds this field in jsdoc, remove warning diagnostics.

These changes may improve user experience, because instead of seeing intrusive hints they will focus on covering their code with error handlers.

@Mahi
Copy link

Mahi commented Oct 10, 2024

I just tried the extension, but immediately uninstalled due to lack of this:

Inside functions, if something throws and the function doesn't have @throws field in its jsdoc comment, show a warning on the function's name. When the user adds this field in jsdoc, remove warning diagnostics.

All I want is to ensure that all throws are documented.

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

No branches or pull requests

2 participants