docgen: Omit unknown type tag from Markdown format output #19571
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related: #18045
Unblocks: #19520
This pull request seeks to resolve an issue where, if
docgen
encounters a type that it cannot parse, the Markdown formatter will output anull
literal value for the type. With these changes, it willsimply not output a type at alloutput text indicating that the type is unknown, in order to avoid misleading the reader into thinking the type isnull
when it is not.This should be considered an interim solution to the larger problem of substituting Doctrine (#18045) in order to use more complex TypeScript types (#18838).
Testing Instructions:
Verify unit tests pass:
Verify there are no changed documentation after running: