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

add possibility to skip the exit confirmation prompt #268

Closed
teto opened this issue Oct 23, 2024 · 2 comments
Closed

add possibility to skip the exit confirmation prompt #268

teto opened this issue Oct 23, 2024 · 2 comments
Labels
done Done, awaiting release enhancement New feature or request

Comments

@teto
Copy link

teto commented Oct 23, 2024

Feature Request

Add possibility of skipping the exit prompt
image
it's an extra key that slows me down

Use Case:

I am iterating over my process-compose.yaml a lot and it keeps asking me wether I want to exit while I am sure I want to exit. Anyhow I use process-compose only for development purposes so there is rarely a case where I dont want to exit.

Proposed Change:

Ideally it would be a flag in ~/.config/process-compose/settings.yaml but I dont mind any solution really, a CLI flag or an environment variable would already be helpful

Who Benefits From The Change(s)?

me ?

Alternative Approaches

add a quit! shortcut one could map in config/process-compose/shortcuts.yaml ?

  quit!:
    description: Force Quit
    shortcut: F10

NB: i am on 1.34.0

@F1bonacc1 F1bonacc1 added the enhancement New feature or request label Oct 26, 2024
@F1bonacc1 F1bonacc1 added the done Done, awaiting release label Nov 2, 2024
@F1bonacc1
Copy link
Owner

Added in v1.40.0

#XDG_CONFIG_HOME/process-compose/settings.yaml

theme: Cobalt
sort:
    by: NAME
    isReversed: false
disable_exit_confirmation: false # if true, will disable the TUI exit confirmation dialog

@teto
Copy link
Author

teto commented Nov 9, 2024

thank you.
If I may, I would say the negative option disable_exit_confirmation is not the most intuitive. exit_confirmation with true as a default would be more straightforward IMO. But I amp happy that a setting exists at all so thanks again !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Done, awaiting release enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants