theckman / yacspin

Yet Another CLi Spinner; providing over 80 easy to use and customizable terminal spinners for multiple OSes
Apache License 2.0
434 stars 13 forks source link

Support stopping a paused spinner #21

Closed theckman closed 4 years ago

theckman commented 4 years ago

It seems reasonable that someone may want a paused spinner to be stopped, without the user seeing another animation or data update. This supports Stop() and StopFail() working on a spinner whose Status() is SpinnerPaused.

codecov[bot] commented 4 years ago

Codecov Report

Merging #21 into master will increase coverage by 0.07%. The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage    92.5%   92.58%   +0.07%     
==========================================
  Files           2        2              
  Lines         387      391       +4     
==========================================
+ Hits          358      362       +4     
  Misses         15       15              
  Partials       14       14
Impacted Files Coverage Δ
spinner.go 92.34% <75%> (+0.08%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 669b9a1...38a3a52. Read the comment docs.