Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/early eval for parameterized options #316

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ppcamp
Copy link

@ppcamp ppcamp commented Sep 21, 2024

Description of changes:

Add support to early eval an option item.

# same as used to be
cat <file=$(ls)>

# same as used to be
cat <file=|__|_$(ls)_|>

# same as used to be
cat <file=|_opt1_|_opt2_|>

# ADDED
cat <file=|_$(ls)_|>

In this case (last scenario), it'll execute ls using cmd/runner/Run function (which keeps record of folder and shell to use), before create the gui, therefore, allowing user to just select one of the output elements.

example

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@RamiAwar
Copy link
Collaborator

Thanks for the contribution! Will take a look at it soon and play around with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

2 participants