Skip to content

Conversation

@xoxys
Copy link
Contributor

@xoxys xoxys commented Mar 24, 2022

Backport: #19201

Unfortunately goldmark 1.4.8 contains a bug whereby html comments on checklist elements are mistakenly rendered. This has been fixed upstream with goldmark 1.4.11.

Regression introduced in: #19127
Upstream issue: yuin/goldmark#285

Fix: #19195

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 24, 2022
@noerw noerw added this to the 1.16.6 milestone Mar 24, 2022
@noerw
Copy link
Member

noerw commented Mar 24, 2022

CI fail is related: Please run 'go mod tidy' and commit the result:

@zeripath
Copy link
Contributor

Let me fix this.

Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
@zeripath
Copy link
Contributor

I've added a testcase to prevent this from happening again.

@xoxys
Copy link
Contributor Author

xoxys commented Mar 24, 2022

Thanks!

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 24, 2022
@zeripath zeripath changed the title Bump goldmark to v1.4.11 Bump goldmark to v1.4.11 (#19201) Mar 24, 2022
@techknowlogick techknowlogick merged commit a13d64b into go-gitea:release/v1.16 Mar 24, 2022
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants