Skip to content

feat(api-markdown-documenter): Add documentation domain to mdast transformation #24787

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

Draft
wants to merge 70 commits into
base: main
Choose a base branch
from

Conversation

Josmithr
Copy link
Contributor

@Josmithr Josmithr commented Jun 8, 2025

Also replace custom Markdown rendering with mdast-util-to-markdown.

@Josmithr Josmithr requested a review from jumyhre June 8, 2025 22:05
@github-actions github-actions bot added base: main PRs targeted against main branch area: website public api change Changes to a public API labels Jun 8, 2025

Test package

# Remarks {#test-suite-a-remarks}

This remarks block includes a bulleted list!

- Bullet 1
\- Bullet 1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: make sure list items don't get escaped

node: HorizontalRuleNode,
context: TransformationContext,
): [MdastThematicBreak] {
// TODO: Do we need to do anything special in tables?
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: add a unit test for this

}

/**
* TODO
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO

import type { TransformationContext } from "../TransformationContext.js";

/**
* TODO
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO

}

/**
* TODO
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: website base: main PRs targeted against main branch public api change Changes to a public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant