New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleanup output when no previous version #13
Conversation
Remove dead link at bottom Don't keep dependency section empty when no dependencies, mention no dependencies but keep section header for consistency. Signed-off-by: Derek McGowan <derek@mcg.dev>
We should also do 1.0 of this project, it would help for that 😆 |
* **{{$dep.Name}}** {{if $dep.Previous}}{{$dep.Previous}} -> {{$dep.Ref}}{{else}}{{$dep.Ref}} **_new_**{{end}} | ||
{{- end}} | ||
{{- else}} | ||
This release has no dependency changes |
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.
Should we omit the whole "dependency changes" section if there's no previous deps?
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 was thinking that, but since it is a top level header, I feel like it is better to say it explicitly. Otherwise you might scroll to the bottom looking for it and wonder if you missed it or was excluded
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.
To me, no dependency changes is a good feature too ;)
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.
Works for me 👍
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.
LGTM
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.
LGTM
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.
LGTM
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.
LGTM
Remove dead link at bottom when no previous version.
Don't keep dependency section empty when no dependencies, mention no dependencies but keep section header for consistency.
This will help use this tool for projects reaching 1.0