-
Notifications
You must be signed in to change notification settings - Fork 393
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.
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(gnokey): in maketx addpkg
, change -deposit
to -send
#3122
base: master
Are you sure you want to change the base?
Conversation
maketx addpkg
, change deposit
to send
maketx addpkg
, change -deposit
to -send
maketx addpkg
, change -deposit
to -send
maketx addpkg
, change -deposit
to -send
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
@leohhhn, can you fix the failed CI tests? |
Can we wait to do this until we decide if we want to rename OrigSend to something like Deposit? |
Removed the |
This PR is stale because it has been open 3 months with no activity. Remove stale label or comment or this will be closed in 3 months. |
Description
Changes the
deposit
flag tosend
. I think this change makes sense from the consistency perspective. I've had multiple times happen where I'm trying to deploy something with some initial balance, trying-send
, only realizing that it's-deposit
in this case.It's quite obvious that the
send
will go to the deployed realm's address, so I think it's a valid change.Contributors' checklist...
BREAKING CHANGE: xxx
message was included in the description