Skip to content

Commit

Permalink
Merge pull request #113 from cpuguy83/fix_codeblock_newline
Browse files Browse the repository at this point in the history
Fix trailing newline in code blocks
  • Loading branch information
cpuguy83 authored Mar 18, 2024
2 parents c2c0656 + fd6bc09 commit 1e915fd
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion md2man/roff.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ const (
codespanTag = "\\fB"
codespanCloseTag = "\\fR"
codeTag = "\n.EX\n"
codeCloseTag = "\n.EE\n"
codeCloseTag = ".EE\n" // Do not prepend a newline character since code blocks, by definition, include a newline already (or at least as how blackfriday gives us on).
quoteTag = "\n.PP\n.RS\n"
quoteCloseTag = "\n.RE\n"
listTag = "\n.RS\n"
Expand Down
8 changes: 6 additions & 2 deletions md2man/roff_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,14 @@ type TestParams struct {
func TestCodeBlocks(t *testing.T) {
tests := []string{
"```\nsome code\n```\n",
".nh\n\n.EX\nsome code\n\n.EE\n",
".nh\n\n.EX\nsome code\n.EE\n",

"```bash\necho foo\n```\n",
".nh\n\n.EX\necho foo\n\n.EE\n",
".nh\n\n.EX\necho foo\n.EE\n",

// make sure literal new lines surrounding the markdown block are preserved as they are intentional
"```bash\n\nsome code\n\n```",
".nh\n\n.EX\n\nsome code\n\n.EE\n",
}
doTestsParam(t, tests, TestParams{blackfriday.FencedCode})
}
Expand Down

0 comments on commit 1e915fd

Please sign in to comment.