#493 linkable versions in release notes, formatting changes #573
Conversation
} | ||
|
||
def "formats header when regular version"() { | ||
expect: | ||
DetailedFormatter.header("v0.1.0", new Date(1483500000000), true) == "# v0.1.0 (2017-01-04) - " | ||
DetailedFormatter.header("v0.1.0", true) == "# v0.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess now emphasized version would be "#", and a regular one "####". What do you think @mockitoguy ?
private static String buildHeader(String version, Date date, String prefix, String postfix) { | ||
return prefix + version + " (" + DateUtil.formatDate(date) + ")" + postfix + " - "; | ||
private static String buildHeader(String version, String prefix, String postfix) { | ||
return prefix + version + postfix; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems that postfix is always "" so it's no longer necessary.
authorsSummaryAppendix(contributions, contributors); | ||
static String releaseSummary(Date date, String version, ContributionSet contributions, Map<String, Contributor> | ||
contributors, String vcsCommitsLink, String publicationRepository) { | ||
return " - " + "(" + DateUtil.formatDate(date) + ") " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
" - " + "(" can be just " - (".
Also the format is different than the one suggested by the issue submitted. It is:
(2017-11-20) n commits
and should be:
2017-11-20 - n commits
Let's put this date prefix into a private method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for such a quick turnaround! Please consider my feedback
Thanks for the contribution! |
My pleasure! |
Changed formatting in release notes to have linkable versions.