-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Upgrade chroma to v2.3.0 #21259
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
Upgrade chroma to v2.3.0 #21259
Conversation
It might be possible to use |
Now it uses chroma's It seems to work fine, but I'm unsure whether that |
…ction chroma.SplitTokensIntoLines, pre-allocate memory for lines
Some small changes:
and LGTM |
make L-G-T-M work |
* upstream/main: [skip ci] Updated translations via Crowdin Add author search input (go-gitea#21246) Upgrade chroma to v2.3.0 (go-gitea#21259)
The behaviour of
PreventSurroundingPre
has changed in alecthomas/chroma#618 so that apparently it now causes line wrapper tags to be no longer emitted, but we need some form of indication to split the HTML into lines, so I did what yuin/goldmark-highlighting#33 did and added thenopWrapper
.Maybe there are more elegant solutions but for some reason, just splitting the HTML string on
\n
did not work.