Skip to content

Report circular JSDoc type references#27404

Merged
sandersn merged 1 commit intorelease-3.1from
js/report-circular-jsdoc-type-references
Oct 5, 2018
Merged

Report circular JSDoc type references#27404
sandersn merged 1 commit intorelease-3.1from
js/report-circular-jsdoc-type-references

Conversation

@sandersn
Copy link
Copy Markdown
Member

JSDoc types references can often be to values, which can often be circular in ways that types tied to declarations cannot. I decided to create a separate property on SymbolLinks rather than reusing declaredType, although I'm not sure that's strictly required.

Fixes #27346

JSDoc types references can often be to values, which can often be
circular in ways that types tied to declarations cannot. I decided to
create a separate property on SymbolLinks rather than reusing
declaredType, although I'm not sure that's strictly required.
@sandersn sandersn added this to the TypeScript 3.1 milestone Sep 27, 2018
Copy link
Copy Markdown
Member

@DanielRosenwasser DanielRosenwasser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh boy

@sandersn sandersn merged commit 36e212b into release-3.1 Oct 5, 2018
@sandersn sandersn deleted the js/report-circular-jsdoc-type-references branch October 5, 2018 22:41
@sandersn
Copy link
Copy Markdown
Member Author

sandersn commented Oct 5, 2018

This is now in master as well.

sandersn added a commit that referenced this pull request Oct 8, 2018
JSDoc types references can often be to values, which can often be
circular in ways that types tied to declarations cannot. I decided to
create a separate property on SymbolLinks rather than reusing
declaredType, although I'm not sure that's strictly required.
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
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.

2 participants