tokuhirom / Minilla

Authorizing tool for CPAN modules
https://metacpan.org/release/Minilla
Other
97 stars 65 forks source link

{{$NEXT}}: describe detailed error #264

Closed cxw42 closed 4 years ago

cxw42 commented 5 years ago

When running minil release, the {{$NEXT}} line in a Changes file must meet all the criteria. This commit adds code to tell the user what the specific problem is if the {{$NEXT}} line does not meet all the criteria.

For example,

  {{$NEXT}}

  1.01 2001-02-03
      Some change

is invalid, because there are no changelog entries under {{$NEXT}}. However, the current code says No mention of {{$NEXT}}, even though {{$NEXT}} is in the file. I believe this change will reduce user confusion.

This PR also includes the fix in #260, so that it can be applied cleanly after #260.

foursixnine commented 4 years ago

@tokuhirom This patch actually helped me (And I guess some others) to figure out where a problem was... Would you consider accepting the PR? cc @skaji :)

syohex commented 4 years ago

I have merged

foursixnine commented 4 years ago

Thanks!