search

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

Rename Config.Delay to Config.Frequency #24

Closed theckman closed 4 years ago

theckman commented 4 years ago

The name "delay" made more sense when the ability to update data was tied to the animation's frequency. Now that data updates happen independently from the spinner animating, it made more sense to make this the animation frequency.

codecov[bot] commented 4 years ago

Codecov Report

Merging #24 into master will increase coverage by 0.07%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
+ Coverage   92.58%   92.65%   +0.07%     
==========================================
  Files           2        2              
  Lines         391      395       +4     
==========================================
+ Hits          362      366       +4     
  Misses         15       15              
  Partials       14       14
Impacted Files Coverage Δ
spinner.go 92.42% <100%> (+0.07%) :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 fd5172c...5060760. Read the comment docs.