sensu / sensu-influxdb-handler

Sensu Go InfluxDB Metrics Handler
https://sensu.io
MIT License
14 stars 16 forks source link

Change how Travis setups and runs goreleaser #26

Closed ccressent closed 5 years ago

ccressent commented 5 years ago

This change is based off the goreleaser manual, see https://goreleaser.com/ci/

While trying to make a new release (3.1.2), Travis' deployment part that uses goreleaser stopped working; the go get github.com/goreleaser/goreleaser command would always fail. This PR changes the way we setup and run goreleaser, based off the goreleaser manual.

The change has been tested by creating a temporary tag (since deleted) to force Travis to run a deployment of this branch. goreleaser was successfully installed and run until it complained that the tag name did not respect semantic versioning. I'm expecting it will just work once I try a proper release with this PR merged.

Before: https://travis-ci.org/sensu/sensu-influxdb-handler/builds/496695795 After: https://travis-ci.org/sensu/sensu-influxdb-handler/builds/496827401

A similar change probably needs to be applied on all our .travis.yml files.