Feat/add custom command

[code-steadfast]

#3

[cuinixam]

Why do we need to wrap the custom command as a pipeline step and not just use the execution context subprocess executor?

You have to consider the fact that the pypeline run shall exit even if the process that was started did not. If I for example use the pypeline run with the "code ." command, I would like that the pypeline execution exits but the VS Code still runs. How is this handled?

[cuinixam]

Hi @code-steadfast, thanks for taking care of this issue. I think we need to cover to separate use cases:

• run a command like uv lock or python --version where I want to wait to the process to run and then exit
• open a GUI like VS Code code where I want to start it and let it run and exit the pypeline execution immediately

I think we need an extra argument for this with the default to wait for the process to finish.

What do you think? Did you test this feature locally?

[code-steadfast]

Hi @cuinixam, Thanks to you for reviewing and I tested locally and its working

.venv\Scripts\python src\pypeline\__run.py run --step CreateVEnv --single --command "python --version"

Regarding extra argument may be I try something like "Popen" option

[code-steadfast]

Hi @cuinixam, Please check these changes.

Thanks 😸

[cuinixam]

I think it is better if we explicitly separate the commands which shall run at the end of the pipeline and an application that shall be started after the pipeline runs.

For example:

• pypeline run --step CreateVEnv --command "echo" will create a step for the echo command and run it after the pipeline ran. The pypeline execution will wait for the command to exit and then finish
• pypeline run --step CreateVEnv --application "code ." will not create any extra step but just run the pipeline and then spawn a new process with the command to start the application and exit immediately

We can discuss if the name application is the best name but otherwise I think it is better to explicitly separate the two use cases as two different arguments.

[cuinixam]

Fixes #3