axiomhq / cli

The power of Axiom on the command line.
https://axiom.co
MIT License
41 stars 11 forks source link

Add '--no-spinner' flag to disable spinner #82

Closed lukasmalkmus closed 3 years ago

lukasmalkmus commented 3 years ago

Adds a global --no-spinner flag to disable the spinner on the command line. This is useful, when tools in a pipe produce output as well, like pv in cat myfile | pv | axiom ingest mydataset.

Closes #81.

codecov[bot] commented 3 years ago

Codecov Report

Merging #82 (0d7a0cd) into main (29145be) will decrease coverage by 0.08%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #82      +/-   ##
==========================================
- Coverage   14.34%   14.25%   -0.09%     
==========================================
  Files           9        9              
  Lines         502      505       +3     
==========================================
  Hits           72       72              
- Misses        415      418       +3     
  Partials       15       15              
Impacted Files Coverage Δ
cmd/axiom/root/root.go 0.00% <0.00%> (ø)
pkg/terminal/io.go 0.00% <0.00%> (ø)

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 29145be...0d7a0cd. Read the comment docs.

bahlo commented 3 years ago

LGTM, thanks for the quick resolution