Skip to content

added fix to the ls command when txtx.yaml file doesn't exist#535

Merged
MicaiahReid merged 2 commits intosolana-foundation:mainfrom
0xzrf:ls_err
Feb 26, 2026
Merged

added fix to the ls command when txtx.yaml file doesn't exist#535
MicaiahReid merged 2 commits intosolana-foundation:mainfrom
0xzrf:ls_err

Conversation

@0xzrf
Copy link
Copy Markdown
Contributor

@0xzrf 0xzrf commented Feb 20, 2026

Description

Running surfpool ls on a folder that doesn't have txtx.yaml file in it(or even provided in cli) doesn't output anything

Steps to Reproduce

  1. Run surfpool ls on a folder that doesn't have txtx.yaml

Current Behavior

The process exits with a non-zero value, but doesn't log anything that might be helpful to the user

Expected Behavior

The command should:

  • Show some error why specifically the command not work

What this PR Solves

This isn't solving a huge problem, but it's still a contribution, as I see it doesn't help the user to know what went wrong. This PR aims to add a simple error handling to the surfpool_cli/src/cli/mod.rs/FN:handle_list_command to show error when something goes wrong while executing the surfpool ls command

@MicaiahReid
Copy link
Copy Markdown
Collaborator

Thanks for the PR, @0xzrf! I changed slightly to more generally improve the error displaying. Will merge once CI runs!

@MicaiahReid MicaiahReid merged commit 3cf8a8c into solana-foundation:main Feb 26, 2026
4 checks passed
@0xzrf 0xzrf deleted the ls_err branch February 27, 2026 09:00
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.

2 participants