Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

[token-cli] Parse compute unit limit with the proper name #7449

Merged
merged 1 commit into from
Nov 4, 2024
Merged

[token-cli] Parse compute unit limit with the proper name #7449

merged 1 commit into from
Nov 4, 2024

Conversation

samkim-crypto
Copy link
Contributor

@samkim-crypto samkim-crypto commented Nov 3, 2024 •
edited
Loading

Problem

It seems that we define compute_unit_limit argument in the clap app, but try to parse it as compute_unit_price.

Summary of Changes

Parse compute_unit_limit properly.

@samkim-crypto samkim-crypto marked this pull request as ready for review November 4, 2024 01:46
joncinque
joncinque approved these changes Nov 4, 2024
View reviewed changes
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Oops, nice find!

@samkim-crypto samkim-crypto merged commit 4374b7d into solana-labs:master Nov 4, 2024
31 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@joncinque joncinque joncinque approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samkim-crypto @joncinque