-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 comment highlighting #1541
Doc comment highlighting #1541
Conversation
Added test.
Overall, this looks good. Obvs there's some conflicts, but I think they're just files to regen. Regex's seem fine, but I'm not really qualified to review them. That said, I think we're comfortable merging now even if I can't and making changes later if we have improvements. |
It's been a while since I last looked at this. |
Alright, I think this is ready now. @mAAdhaTTah You might want to take a look at the changes I made. |
This PR implements #1475.
It adds support for JavaDoc (Java), PHPDoc (PHP), and JSDoc (JS, JSX, N4JS, TS, TSX).
This will also affect languages which include any of the above languages.
Notes:
Examples
TypeScript
JavaScript
Java
PHP