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

Doc for ignoreTryCatch incorrectly parsed by IDE #1369

Closed
1 of 4 tasks
ClementValot opened this issue Dec 7, 2022 · 3 comments
Closed
1 of 4 tasks

Doc for ignoreTryCatch incorrectly parsed by IDE #1369

ClementValot opened this issue Dec 7, 2022 · 3 comments

Comments

@ClementValot
Copy link
Contributor

ClementValot commented Dec 7, 2022

  • Rollup Plugin Name: commonjs
  • Rollup Plugin Version: ^23.0.3

Documentation Is:

  • Missing
  • Needed
  • Confusing
  • Not Sure?

Please Explain in Detail...

The source of many a headache, the ignoreTryCatch option defaults to true according to the code
image
But when hovering the options for commonjs, Webstorm (2022.3 here, but was the same in previous versions) indicates that it defaults to false
image

Very unsure why Webstorm would incorrectly parse the documentation, is it a Webstorm bug, or an inconsistency somewhere in the declaration files?

@shellscape
Copy link
Collaborator

This is an issue with the types, as seen here.

Please feel free to open a PR to correct

@ClementValot
Copy link
Contributor Author

It didn't even occur to me that it would be a tag in the doc -_-

Following PR changes the tag and corrects a few typos in the docs
#1370

@ClementValot
Copy link
Contributor Author

Docs clarified in the above PR

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