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.

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

Make keys generate to no longer fund #1887

Merged
merged 6 commits into from
Mar 6, 2025
Merged

Make keys generate to no longer fund #1887

merged 6 commits into from
Mar 6, 2025

Conversation

Ifropc
Copy link
Contributor

@Ifropc Ifropc commented Feb 22, 2025

What

Previously, --fund flag was added, and funding by default gave a warning.
In this PR -no-fund flag was removed, and default behavior flipped to no longer fund

Why

Closing #1407

Known limitations

Depends on #1888

Copy link
Member

@fnando fnando left a comment

Choose a reason for hiding this comment

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

lgtm!

@Ifropc Ifropc enabled auto-merge (squash) March 6, 2025 00:33
@Ifropc Ifropc merged commit 01d1d36 into main Mar 6, 2025
32 checks passed
@Ifropc Ifropc deleted the keys-generate-no-fund branch March 6, 2025 01:02
@tupui
Copy link

tupui commented Mar 6, 2025

Oh, no deprecation warning? I was personally relying on auto funding for my tests 😅

@fnando
Copy link
Member

fnando commented Mar 6, 2025

@tupui this command was already outputing a deprecation message on v22, and will be removed on v23.

$ stellar --version
stellar 22.3.0 (3a00de5e6c579479e943f5914f2fcfaca8426c8c)
stellar-xdr 22.1.0 (e13922970800d95b523413018b2279df42df3442)
xdr curr (529d5176f24c73eeccfa5eba481d4e89c19b1181)

 $ stellar keys generate foo
⚠️ Behavior of `generate` will change in the future, and it will no longer fund by default. If you want to fund please provide `--fund` flag. If you don't need to fund your keys in the future, ignore this warning. It can be suppressed with -q flag.
✅ Key saved with alias foo in "/private/tmp/test/.stellar/identity/foo.toml"
✅ Account KeyName("foo") funded on "Test SDF Network ; September 2015"

You can fund the account by using stellar keys generate foo --fund from now on.

@tupui
Copy link

tupui commented Mar 6, 2025

Thanks, I will pay more attention to the command outputs 👍 It was missing from the release notes of v22 which I read. That's how I came across this PR.

[EDIT] I see Add warning on no fund flag but this is really not clear, especially without a deprecated section.

@fnando
Copy link
Member

fnando commented Mar 6, 2025

@tupui good call! I'll let the team know so we have better documentation on deprecations and removals using the release page.

@tupui
Copy link

tupui commented Mar 6, 2025

Thank you 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[22.0] stellar keys generate should no longer fund
3 participants