Skip to content

MarkdownPart: move to separate mdoc-parser module#812

Merged
tgodzik merged 1 commit intoscalameta:mainfrom
kitbellew:812
Nov 3, 2023
Merged

MarkdownPart: move to separate mdoc-parser module#812
tgodzik merged 1 commit intoscalameta:mainfrom
kitbellew:812

Conversation

@kitbellew
Copy link
Copy Markdown
Contributor

@kitbellew kitbellew commented Oct 31, 2023

scalafmt has an own markdown parser, to be able to format any scala code found in fenced blocks. In addition to duplicating functionality in that parser, it also doesn't support indented code blocks.

Let's instead extract the parser portion of the mdoc logic into a module so that it can be re-used. That module will have no dependencies.

Helps with scalameta/scalafmt#3672.

scalafmt has an own markdown parser, to be able to format any scala code
found in fenced blocks. In addition to duplicating functionality in that
parser, it also doesn't support indented code blocks.

Let's instead extract the parser portion of the mdoc logic into a module
so that it can be re-used. That module will have no dependencies.
@kitbellew kitbellew requested review from keynmol and tgodzik October 31, 2023 03:21
@kitbellew
Copy link
Copy Markdown
Contributor Author

@tgodzik fyi, in case you missed this :)

Copy link
Copy Markdown
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

LGTM!

@tgodzik
Copy link
Copy Markdown
Contributor

tgodzik commented Nov 3, 2023

One minor thing is due to the last update we need to release on JDK 11 now.

@tgodzik tgodzik merged commit b8b3fd7 into scalameta:main Nov 3, 2023
@kitbellew kitbellew deleted the 812 branch November 9, 2023 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants