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 enter_accept to immediately execute an accepted command #1311

Merged
merged 6 commits into from Oct 20, 2023
Merged

Conversation

ellie
Copy link
Member

@ellie ellie commented Oct 19, 2023

enter_accept will make Atuin immediately accept an execute a command
when selected. It defaults to false in our binary, but the default
config enables it.

This means that users who already use atuin will not default to the new
behaviour unless they opt in, but new users will have it by default.

Thanks to @davidhewitt for the patch and bulk of this implementation!
Currently we have it just for zsh, but I'll follow up with other shells
(unless anyone beats me to it :D)

This also uses "tab" to select and edit a command

davidhewitt and others added 3 commits October 19, 2023 20:15
enter_accept will make Atuin immediately accept an execute a command
when selected. It defaults to false in our binary, but the default
config enables it.

This means that users who already use atuin will not default to the new
behaviour unless they opt in, but new users will have it by default.

Thanks to @davidhewitt for the patch and bulk of this implementation!
Currently we have it just for zsh, but I'll follow up with other shells
(unless anyone beats me to it :D)
@vercel
Copy link

vercel bot commented Oct 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
atuin-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 20, 2023 6:17am

@ellie
Copy link
Member Author

ellie commented Oct 19, 2023

Ref: #817 (comment)

conradludgate
conradludgate previously approved these changes Oct 20, 2023
docs/docs/config/config.md Outdated Show resolved Hide resolved
Co-authored-by: Conrad Ludgate <conradludgate@gmail.com>
@ellie ellie merged commit 88f3e2a into main Oct 20, 2023
9 of 10 checks passed
@ellie ellie deleted the ellie/exec branch October 20, 2023 06:16
@ellie
Copy link
Member Author

ellie commented Oct 20, 2023

@davidhewitt thanks again for this! If you fancy a shiny contributor-only sticker in the post, please fill out this form!

@davidhewitt
Copy link
Contributor

Thanks! Not sure I did a huge amount, but I filled out the form anyway 👀

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

Successfully merging this pull request may close these issues.

None yet

3 participants