Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: auto-update ADDR in Makefile #63

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore: auto-update ADDR in Makefile #63

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4d9c5eb
chore: auto-update ADDR
amessbee Sep 25, 2024
f68137a
fixup! remove unnecessary instructions
amessbee Sep 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 14 additions & 3 deletions contract/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,20 @@ BLD=000000ubld
ATOM_DENOM=ibc/BA313C4A19DFBF943586C0387E6B11286F9E416B4DD27574E6909CABE0E342FA
ATOM=000000$(ATOM_DENOM)

# your deployer address made from make add-address
# ADDR=agoric1k78s7qz7rxy8afyjrqk3dntg8m83zaw3upe60p
ADDR=agoric125d8s0tamx50gpqdd3m7f5gtkzkcnjs0ehsujr

# fetch the address of the deployer account from the chain - you may need to adjust this to match your setup
Copy link
Member

Choose a reason for hiding this comment

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

Why would we need to adjust this?
Isn't the point of this pr to take care of that?

USERNAME ?= user1

ADDR := $(shell kubectl exec -i agoriclocal-genesis-0 -c validator -- sh -c '\
if ! agd keys show $(USERNAME) >/dev/null 2>&1; then \
agd keys add $(USERNAME) --output json; \
else \
agd keys show $(USERNAME) --output json; \
fi' | jq -r ".address" \
)

$(info USERNAME is $(USERNAME))
$(info ADDR is $(ADDR))
Copy link
Member

Choose a reason for hiding this comment

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

Please update (remove) the corresponding instructions in the README


PROVISION_POOL_ADDR=agoric1megzytg65cyrgzs6fvzxgrcqvwwl7ugpt62346

Expand Down
Loading