Skip to content

Conversation

beorn7
Copy link
Member

@beorn7 beorn7 commented Oct 15, 2019

This also fixes a capitalization issue in the godoc examples for the API. See commit description.

Interestingly, no breaking changes in the API client. Is it approaching stability? :o)

The part after the underscore has to be lowercase. Otherwise, it is
seen as the name of a Go type, which doesn't exist, which will result
in the example not showing up in godoc.

Signed-off-by: beorn7 <beorn@grafana.com>
Just to make sure we are up to date before releasing.
This only affects golang.org/x/sys.

Signed-off-by: beorn7 <beorn@grafana.com>
One might argue that supporting the newest Pushgateway is just an
enhancement or even a bugfix (and this should be v1.1.1, which would
be a really nice version number). I decided to go for a positive view
of things and call it a feature. If something else changes its
behavior, it's not really a bug from client_golang's point of view.

Signed-off-by: beorn7 <beorn@grafana.com>
Copy link
Member

@csmarchbanks csmarchbanks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Good catch on the example names.

@krasi-georgiev
Copy link
Contributor

LGTM

@beorn7 beorn7 merged commit 9a2ab94 into master Oct 15, 2019
@beorn7 beorn7 deleted the beorn7/release branch October 15, 2019 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants