cmd/relnote: include closing dd tags in HTML output
In HTML5, closing dd tags are optional, meaning it is okay to either
include them or exclude them. For the purpose of writing the release
notes HTML file, which involves many people collaborating, it's less
costly to always include optional HTML tags.
This can eliminate the need for doing so manually, like in CL 217701,
CL 245979, and so on.
Change-Id: Idc16ea61e2b9624d1aa48543df2b913936993909
Reviewed-on: https://go-review.googlesource.com/c/build/+/260497
Trust: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
diff --git a/cmd/relnote/relnote.go b/cmd/relnote/relnote.go
index 67b106e..ebde350 100644
--- a/cmd/relnote/relnote.go
+++ b/cmd/relnote/relnote.go
@@ -119,16 +119,16 @@
if strings.HasPrefix(pkg, "cmd/") {
continue
}
- fmt.Printf("<dl id=%q><dt><a href=%q>%s</a></dt>\n <dd>\n",
+ fmt.Printf("<dl id=%q><dt><a href=%q>%s</a></dt>\n <dd>",
pkg, "/pkg/"+pkg+"/", pkg)
for _, change := range changes[pkg] {
changeURL := fmt.Sprintf("https://golang.org/cl/%d", change.CL.Number)
subj := clSubject(change.CL)
subj = strings.TrimPrefix(subj, pkg+": ")
- fmt.Printf(" <p><!-- CL %d -->\n TODO: <a href=%q>%s</a>: %s\n </p>\n\n",
+ fmt.Printf("\n <p><!-- CL %d -->\n TODO: <a href=%q>%s</a>: %s\n </p>\n",
change.CL.Number, changeURL, changeURL, html.EscapeString(subj))
}
- fmt.Printf("</dl><!-- %s -->\n\n", pkg)
+ fmt.Printf(" </dd>\n</dl><!-- %s -->\n\n", pkg)
}
} else {
