Skip to content

In quick info, show all jsdoc tags#19357

Merged
1 commit merged into
masterfrom
quickInfoJsDocTags
Oct 30, 2017
Merged

In quick info, show all jsdoc tags#19357
1 commit merged into
masterfrom
quickInfoJsDocTags

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Oct 19, 2017

Fixes #19284

@ghost ghost requested a review from armanio123 October 19, 2017 22:03
@ghost ghost force-pushed the quickInfoJsDocTags branch 3 times, most recently from b735494 to d67aefb Compare October 20, 2017 15:35
@ghost ghost force-pushed the quickInfoJsDocTags branch from d67aefb to 69177d2 Compare October 20, 2017 17:29
@sandersn
Copy link
Copy Markdown
Member

Check with @minestarks before merging this. She designed this API for a specific use-case which only wanted to see non-standard tags. I can't remember the case, but it will be broken if standard tags start being included.

@minestarks
Copy link
Copy Markdown
Member

@sandersn @Andy-MS actually standard tags were OK in my use case - I just ignore them. As long as I'm still getting nonstandard ones it's fine.

@ghost ghost merged commit 212efd5 into master Oct 30, 2017
@ghost ghost deleted the quickInfoJsDocTags branch October 30, 2017 17:27
@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants