-
Notifications
You must be signed in to change notification settings - Fork 164
Conversation
Codecov Report
@@ Coverage Diff @@
## master #53 +/- ##
=======================================
Coverage 78.59% 78.59%
=======================================
Files 8 8
Lines 271 271
=======================================
Hits 213 213
Misses 49 49
Partials 9 9 Continue to review full report at Codecov.
|
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.
2 questions, otherwise LGTM and let's release (:
CHANGELOG.md
Outdated
|
||
### Removed | ||
|
||
n/a |
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.
n/a? What do you mean? That we don't have changelog for below and 1.1.0 or what?
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.
if that is the case, can we actually put 1.1 and 1.0 headers with n/a mention (or more verbose version "changelog started from 1.2")
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.
n/a stands for not applicable, because there was nothing removed in this release
CHANGELOG.md
Outdated
|
||
## [Unreleased] | ||
|
||
## [1.2.0] - 2018-06-04 |
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.
worth to add actual link to (non-existing yet) 1.2.0 release on github (https://github.com/grpc-ecosystem/go-grpc-prometheus/releases/tag/v1.2)
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 definitely agree this comment.
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! This is perfect. 👍
Closes #52 #43
@Bplotka