franciscolourenco / done

A fish-shell package to automatically receive notifications when long processes finish.
MIT License
766 stars 70 forks source link

Use `fish_postexec` instead of `fish_prompt` #130

Closed wesalvaro closed 9 months ago

wesalvaro commented 2 years ago

Using the fish_postexec event removes the reliance on history as the command that just finished executing is passed in as the first argument to the function. The fish_prompt event is also fired when we don't need to run.

franciscolourenco commented 9 months ago

fish_postexec was introduced in fish 2.2.0 (released July 12, 2015), so safe to use. Thank you for the PR!