Skip to content

Conversation

Songmu
Copy link
Collaborator

@Songmu Songmu commented Aug 20, 2025

Goldmark handles raw br tags, so we don't need to process them ourselves.

In addition, this unnecessary processing prevented us from showing examples of br tags in the text, so I fixed that as well.

Copy link
Contributor

Code Metrics Report

main (b58cb77) #379 (e2e4aad) +/-
Coverage 31.7% 31.7% -0.1%
Code to Test Ratio 1:0.8 1:0.8 +0.0
Test Execution Time 9s 11s +2s
Details
  |                     | main (b58cb77) | #379 (e2e4aad) |  +/-  |
  |---------------------|----------------|----------------|-------|
- | Coverage            |          31.7% |          31.7% | -0.1% |
  |   Files             |             26 |             26 |     0 |
  |   Lines             |           3054 |           3053 |    -1 |
- |   Covered           |            971 |            970 |    -1 |
+ | Code to Test Ratio  |          1:0.8 |          1:0.8 |  +0.0 |
  |   Code              |           6270 |           6267 |    -3 |
  |   Test              |           5464 |           5464 |     0 |
- | Test Execution Time |             9s |            11s |   +2s |

Code coverage of files in pull request scope (75.8% → 75.8%)

Files Coverage +/- Status
md/md.go 75.8% -0.1% modified

Reported by octocov

@Songmu Songmu merged commit f115097 into main Aug 20, 2025
1 check passed
@Songmu Songmu deleted the br branch August 20, 2025 06:28
@github-actions github-actions bot mentioned this pull request Aug 20, 2025
@Songmu Songmu added the bug Something isn't working label Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant