public
/
icepick
5
0
Fork 0
Code Issues 18 Pull Requests 1 Packages Projects Releases Wiki Activity

Tracker for CLI improvements #14

New Issue
Closed
opened 2024-11-26 11:43:40 +00:00 by ryan · 3 comments
ryan commented 2024-11-26 11:43:40 +00:00
Owner
Copy Link
No description provided.
ryan added this to the Custody Framework project 2024-11-26 11:43:40 +00:00
ryan commented 2024-11-26 11:47:51 +00:00
Author
Owner
Copy Link

Currently, arguments are either positional and therefore required, or --long-args and therefore optional. However, while writing the SOL transfer, I noticed it gets rather bulky when using positional required arguments. For example:

icepick sol transfer <amount> <to-address> <recent-blockhash> <from-address> --from-account <account> --fee-payer <fee-payer-account> --fee-payer-address <fee-payer-address>

This should instead probably be represented as:

icepick sol transfer <amount> <to-address> --recent-blockhash <blockhash> --from-address <address> --from-account [account] --fee-payer [account] --fee-payer-address [account]

Amount, to-address, blockhash, and from-address are still reuqired, but only amount and to address (the two "actually" relevant values) are positional.

Currently, arguments are either positional and therefore required, or `--long-args` and therefore optional. However, while writing the SOL transfer, I noticed it gets rather bulky when using positional required arguments. For example: ```sh icepick sol transfer <amount> <to-address> <recent-blockhash> <from-address> --from-account <account> --fee-payer <fee-payer-account> --fee-payer-address <fee-payer-address> ``` This should instead probably be represented as: ```sh icepick sol transfer <amount> <to-address> --recent-blockhash <blockhash> --from-address <address> --from-account [account] --fee-payer [account] --fee-payer-address [account] ``` Amount, to-address, blockhash, and from-address are still reuqired, but only amount and to address (the two "actually" relevant values) are positional.
ryan commented 2024-11-26 11:48:02 +00:00
Author
Owner
Copy Link

Or, make everything --long-args.

Or, make everything `--long-args`.
ryan added this to the Icepick v0.1.0 milestone 2024-12-12 20:30:26 +00:00
ryan commented 2025-02-14 03:45:05 +00:00
Author
Owner
Copy Link

Workflows replace the short and long addresses by having everything be long addresses. Options are marked as required by their description.

Workflows replace the short and long addresses by having everything be long addresses. Options are marked as required by their description.
ryan closed this issue 2025-02-14 03:45:05 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
anton/refactor-toml-to-yaml
ryansquared/spacemesh
anton/error-logging
ryansquared/sol/durable-nonces
ryansquared/icepick-internal
merge-blob-and-values
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Icepick v0.1.0
Projects
Clear projects
No project
Custody Framework
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: public/icepick#14
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?