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 Request: option to not run scripts after insertion #414

Open
dimateos opened this issue May 12, 2023 · 3 comments
Open

Feature Request: option to not run scripts after insertion #414

dimateos opened this issue May 12, 2023 · 3 comments

Comments

@dimateos
Copy link

Hey, thanks for the great extension!

Is your feature request related to a problem? Please describe.
The scripts functionality is very useful, but I feel like the fact that the snippets are always auto run is a limiting factor. When a command is very similar, it's easier to have a single snippet and tune it before runnning, e.g. here I dont want two snippets just to remove "-no":

"terminalAllInOne.scripts": [
    {
        "name": "git: consider unchaged",
        "script": ["git update-index --no-assume-unchanged"],
    },
]

Describe the solution you'd like
Probably a key added to the json object toggling the behavior:

"terminalAllInOne.scripts": [
    {
        "name": "git: consider unchaged",
        "script": ["git update-index --no-assume-unchanged"],
        "run": false,
    },
]

Additional context
This also seems useful for scripts you need to complete, e.g. like a final argument etc.

@github-actions
Copy link

Thank you for contributing to terminal-all-in-one!

@marktenney
Copy link

I came to request this exact same feature and was pleased to see this already posted. I've been adding placeholders into my scripts to prevent them from running on insertion, just so I can let the system try to run it and fail, then hit the up arrow to queue up the command again and edit it before running. Commands should just be entered and not run.

@marktenney
Copy link

@YashTotale Have you given this any thought? It would be great to either have a setting to disable autorun or the ability to set specific scripts to not run automatically.

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

No branches or pull requests

2 participants