Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

improve validator build ux #1011

Merged
merged 2 commits into from
Aug 27, 2021
Merged

improve validator build ux #1011

merged 2 commits into from
Aug 27, 2021

Conversation

evanmcc
Copy link
Contributor

@evanmcc evanmcc commented Aug 27, 2021

rebar3's versioning leaves something to be desired at present and we don't have the resources to fix it ourselves right now. this should make working around it a bit easier.

@@ -11,8 +11,10 @@ MINER_REGISTRY_NAME="$REGISTRY_HOST/team-helium/$REGISTRY_NAME"
DOCKER_NAME="$(basename $(pwd))-${IMAGE_ARCH}_testnet_${BUILDKITE_TAG}"
DOCKERFILE_NAME=".buildkite/scripts/Dockerfile-testval-${IMAGE_ARCH}"

VERSION=$(git describe --abbrev=0 | sed -e 's,validator,,')
Copy link
Contributor

Choose a reason for hiding this comment

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

IIRC this tag is just val not all spelled out, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

val is for testnet, validator is for mainnet, I am a terrible hack

@jadeallenx
Copy link
Contributor

lgtm 👍

@evanmcc evanmcc merged commit 9f67694 into master Aug 27, 2021
@evanmcc evanmcc deleted the pevm/fix-val-releases branch August 27, 2021 18:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants