franciscolourenco / done

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

Specify whether the command finished successfully. #16

Closed skorokithakis closed 7 years ago

skorokithakis commented 7 years ago

Perhaps the notification can say whether the command finished with an error or successfully? This could be as simple as "Finished in X" and "Errored in X".

franciscolourenco commented 7 years ago

Good idea. Probably not "Errored' though.

"To use "errored" you would have to consider error a verb, but the dictionary doesn't. "Error" is a noun, used to mean mistake. You wouldn't say "The program mistaked at line 44." Your usage "threw an error" is correct, because it uses error as a noun."

skorokithakis commented 7 years ago

Yeah, I realized that as I was typing it, but all the alternatives felt worse. "Finished in X" "Finished with an error in X" is too long for the failure case, but maybe adding an extra line with "There was an error" at the end is good enough?

franciscolourenco commented 7 years ago

Finished in X with errors doesn't seem that long:

screen shot 2017-02-12 at 18 19 02

Other options would be Failed in X: screen shot 2017-02-12 at 18 20 10

jorgebucaran commented 7 years ago

First one sounds more natural.

skorokithakis commented 7 years ago

Agreed, that seems fine to me too.

franciscolourenco commented 7 years ago

What do you guys think about Finished in 2m 32s vs Done in 2m 32s ?

screen shot 2017-02-12 at 18 50 52

First seems more natural but second is shorter and playful?

jorgebucaran commented 7 years ago

Definitely done.

franciscolourenco commented 7 years ago

And for errors?

screen shot 2017-02-12 at 18 59 23

😄

franciscolourenco commented 7 years ago

I wonder if Overdone after 2m 32s is clear enough or if Burned after 2m 32s would be better. Also we could add the exit code in case of error Overdone (1) after 2m 32s

skorokithakis commented 7 years ago

I think that's getting a bit too cryptic (not to mention that the metaphor doesn't work because the command took less time than it should :P).

franciscolourenco commented 7 years ago

hihi, so what, keep it to Done in X with errors?

skorokithakis commented 7 years ago

Sounds good to me, or "Finished with errors in X".

franciscolourenco commented 7 years ago

For errors ended up with something like Exit status 1 after 3m 23s.

Done in https://github.com/fisherman/done/releases/tag/1.2.0