Skip to content

Conversation

@aaronbog
Copy link
Collaborator

@aaronbog aaronbog commented May 20, 2025

This pull request contains three main parts.

  1. an implementation of the shell called execute.py
  • This is a function that only concerns itself with executing commands on the host system
  • It facilitates the use of hooks to do additional work without blocking the execution of the program
  • The newly returned data type allows for more functionality over the previous implementation
  1. hooks
  • a way of working with the datastreams from commands
  • allows for async logging
  • better extraction of data in the middle of using a command
  • can be used in a variety of ways and is customizable and extendable
  1. a port of the old shell_out functions
  • This is backward compatible in functionality (not all logging options are currently supported)
  • This can be toggled off instantly should there be any need to revert to the old system

@aaronbog aaronbog marked this pull request as draft May 20, 2025 19:01
@aaronbog aaronbog added enhancement New feature or request shellout Changes to the internal of executing something on the shell; planned labels May 20, 2025
Bogaert Aaron and others added 25 commits May 21, 2025 09:50
Signed-off-by: Bogaert Aaron <[email protected]>
Bogaert Aaron added 29 commits June 10, 2025 11:19
Signed-off-by: Bogaert Aaron <[email protected]>
Signed-off-by: Bogaert Aaron <[email protected]>
Signed-off-by: Bogaert Aaron <[email protected]>
Signed-off-by: Bogaert Aaron <[email protected]>
Signed-off-by: Bogaert Aaron <[email protected]>
Signed-off-by: Bogaert Aaron <[email protected]>
@aaronbog aaronbog changed the title Add a new shell_out that works with AST-commands Add a new shell_out with support for hooks to create async functionality. Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request planned shellout Changes to the internal of executing something on the shell;

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant