Change delimiter in test "sed" command to ':' (#8108)

The current delimiter is `/` which breaks when a branch name contains
that character. `:` is disallowed in branch names so should serve as a
suitable delimiter

This is the same as https://github.com/grafana/loki/pull/8107 but the
branch name doesn't include a `/`. That shouldn't be an issue after this
is merged
pull/8109/head
Travis Patterson 3 years ago committed by GitHub
parent 3f9b9cc961
commit 9322ed0a68
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      Makefile

@ -289,7 +289,7 @@ lint:
########
test: all
$(GOTEST) -covermode=atomic -coverprofile=coverage.txt -p=4 ./... | sed "s/$$/ ${DRONE_STEP_NAME} ${DRONE_SOURCE_BRANCH}/" | tee test_results.txt
$(GOTEST) -covermode=atomic -coverprofile=coverage.txt -p=4 ./... | sed "s:$$: ${DRONE_STEP_NAME} ${DRONE_SOURCE_BRANCH}:" | tee test_results.txt
compare-coverage:
./tools/diff_coverage.sh $(old) $(new) $(packages)

Loading…
Cancel
Save