| Commit message (Collapse) | Author |
|
This commit also remove another load of unneeded features.
|
|
They will not be marked by rustc/cargo as unused, and as atuin doesn't
expose an API all of them _should_ be `pub(crate)`
|
|
That helps remove duplicated code and rustc/cargo will now also show
dead code correctly.
|
|
This fixes the `atuin update` command on Windows.
Windows doesn't let you overwrite a running exe, but it lets you rename
it. This PR special-cases the official `update` plugin by renaming the
running `atuin.exe` to `atuin.old` before the update, and rolling it
back if the update fails.
Note that the `atuin.old` file is left behind on success, which
shouldn't be a problem in practice: it will be overwritten on the next
call to `atuin update` (also deleted if there's no update available),
and is located in `~/.atuin/bin`, which is an isolated location specific
to Atuin.
Fixes #3451
## Checks
- [x] I am happy for maintainers to push small adjustments to this PR,
to speed up the review cycle
- [x] I have checked that there are no existing pull requests for the
same thing
|
|
* feat: add info for 'official' plugins
* fix default features
|
|
* Run 'atuin-<subcmd>' if present when a given subcommand is not recognized
* Send errors to stderr
* Use String instead of OsString for external subcommands
* Remove unused import
* Move external subcommand handling up a level
* Clippy
|