Skip to content

Commit

Permalink
Pin the goreleaser version for Go version support (#3674)
Browse files Browse the repository at this point in the history
* Pin the goreleaser version for Go version support

Without this change, goreleaser@latest requires a newer version of Go
than is currently supported by the project.  Here we pin the version.

* Pin the version of the tools image
  • Loading branch information
zalegrala authored May 14, 2024
1 parent d8c69d4 commit 868d066
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,7 @@ update-mod:

### Release (intended to be used in the .github/workflows/release.yml)
$(GORELEASER):
go install github.com/goreleaser/goreleaser@latest
go install github.com/goreleaser/goreleaser@v1.25.1

.PHONY: release
release: $(GORELEASER)
Expand Down
3 changes: 2 additions & 1 deletion build/tools.mk
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,11 @@ TOOL_DIR ?= tools
TOOL_CONFIG ?= $(TOOL_DIR)/tools.go

TOOLS_IMAGE ?= grafana/tempo-ci-tools
TOOLS_IMAGE_TAG ?= main-d8c69d4

GOTOOLS ?= $(shell cd $(TOOL_DIR) && go list -e -f '{{ .Imports }}' -tags tools |tr -d '[]')

TOOLS_CMD = docker run --rm -t -v ${PWD}:/tools $(TOOLS_IMAGE)
TOOLS_CMD = docker run --rm -t -v ${PWD}:/tools $(TOOLS_IMAGE):$(TOOLS_IMAGE_TAG)

.PHONY: tools-image-build
tools-image-build:
Expand Down

0 comments on commit 868d066

Please sign in to comment.