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

[Java.Interop.Tools.JavaSource] Replace TODO with To be added #937

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

pjcollins
Copy link
Member

@pjcollins pjcollins commented Jan 4, 2022

Partially fixes: #907

Replaces [TODO: @tag] values in generated docs with To be added to
better mirror mdoc behavior for missing elements.

Inline {@value } tags that contain an argument have been updated to
display the argument in a <c/> block. These should be fixed to use
an appropriate CREF element in the future, see #843.

The last remaining [TODO: @{docRoot}] output will be fixed as part of
#930.

Partially fixes: dotnet#907

Replaces `[TODO: @tag]` values in generated docs with `To be added` to
better mirror mdoc behavior for missing elements.

Inline `{@value }` tags that contain an argument have been updated to
display the argument in a `<c/>` block.  These should be fixed to use
an appropriate CREF element in the future, see dotnet#843.
@pjcollins pjcollins requested review from jonpryor and jpobst January 5, 2022 17:30
@jonpryor jonpryor merged commit 4cbc07a into dotnet:main Jan 6, 2022
@pjcollins pjcollins deleted the todo-to-added branch January 6, 2022 16:59
@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2024
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.

[Java.Interop.Tools.JavaSource] Don't emit [TODO: * ] in summaries
2 participants