Skip to content

fix: Format message in JSON format for missing part in wasp-cli#758

Merged
howjmay merged 2 commits intoiotaledger:developfrom
howjmay:wasp-cli
Nov 25, 2025
Merged

fix: Format message in JSON format for missing part in wasp-cli#758
howjmay merged 2 commits intoiotaledger:developfrom
howjmay:wasp-cli

Conversation

@howjmay
Copy link
Member

@howjmay howjmay commented Nov 4, 2025

No description provided.

re := regexp.MustCompile(`Gas fee charged:\s*(\d+)`)

for _, line := range outs {
matches := re.FindStringSubmatch(line)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Now that our cli supports structured json outputs, that shouldnt be needed anymore

Copy link
Member Author

Choose a reason for hiding this comment

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

I think not all the ### cli commands accept --json. I am solving it

@howjmay howjmay force-pushed the wasp-cli branch 7 times, most recently from dd50c2f to b342237 Compare November 18, 2025 10:35
@howjmay howjmay changed the title test: fix wasp-cli_test TestWaspCLIDeposit fix: Format message in JSON format for missing part in wasp-cli Nov 18, 2025
Copy link
Contributor

@cwarnerdev cwarnerdev left a comment

Choose a reason for hiding this comment

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

TestEVMGasPriceMismatch fails but not due to this PR

@howjmay howjmay merged commit 6534429 into iotaledger:develop Nov 25, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants