Skip to content

fix: output action properties at completion of command runner#62

Merged
nats merged 3 commits intobinarylane:mainfrom
nats:fix/action-output
Apr 4, 2025
Merged

fix: output action properties at completion of command runner#62
nats merged 3 commits intobinarylane:mainfrom
nats:fix/action-output

Conversation

@nats
Copy link
Copy Markdown
Contributor

@nats nats commented Mar 5, 2025

Currently ActionRunner when used without --async does not produce any output, making it impossible to see result of bl server action uptime command.

This PR changes behaviour of ActionRunner so that:

  • it prints the action's result_data (if any) as part of the final progress message on stderr
  • any of --output table/plain/tsv/json can be used to print the completed action to stdout

@nats nats force-pushed the fix/action-output branch from ce60819 to c5e1dbd Compare March 5, 2025 07:21
@nats nats marked this pull request as draft March 5, 2025 07:36
@nats nats force-pushed the fix/action-output branch from ed7c797 to b115c4b Compare March 5, 2025 07:39
@nats nats force-pushed the fix/action-output branch from b115c4b to 7154617 Compare March 5, 2025 07:56
@nats nats marked this pull request as ready for review April 4, 2025 03:56
@nats nats merged commit f1a1213 into binarylane:main Apr 4, 2025
18 checks passed
@nats nats deleted the fix/action-output branch April 4, 2025 04:03
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.

1 participant