timescale / tsbs

Time Series Benchmark Suite, a tool for comparing and evaluating databases for time series data
MIT License
1.29k stars 300 forks source link

Update README to reflect the name change of the scale-var flag #34

Closed LeeHampton closed 5 years ago

LeeHampton commented 5 years ago

@RobAtticus @sunsingerus Someone in our support channel caught this yesterday -- one place we forgot to update the -scale-var flag name.

codecov-io commented 5 years ago

Codecov Report

Merging #34 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   61.91%   61.91%           
=======================================
  Files          60       60           
  Lines        2610     2610           
=======================================
  Hits         1616     1616           
  Misses        984      984           
  Partials       10       10

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 8014db8...2133227. Read the comment docs.

sunsingerus commented 5 years ago

Yes indeed, documentation is the thing which is often forgotten

LeeHampton commented 5 years ago

Turns out there were a few places in the code that also referred to the old var name, namely error strings and comments -- updated the PR.