axiomhq / cli

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

Enhancements #104

Closed lukasmalkmus closed 2 years ago

lukasmalkmus commented 2 years ago

A larger PR with some enhancements as I had to catch up on some stuff:

codecov[bot] commented 2 years ago

Codecov Report

Merging #104 (e8b700c) into main (0468ca2) will increase coverage by 0.27%. The diff coverage is 21.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   14.20%   14.47%   +0.27%     
==========================================
  Files           9       10       +1     
  Lines         507      518      +11     
==========================================
+ Hits           72       75       +3     
- Misses        419      427       +8     
  Partials       16       16              
Impacted Files Coverage Δ
pkg/terminal/epipe_other.go 0.00% <0.00%> (ø)
pkg/terminal/io.go 0.00% <0.00%> (ø)
cmd/axiom/main.go 19.23% <75.00%> (+4.94%) :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 0468ca2...e8b700c. Read the comment docs.

lukasmalkmus commented 2 years ago

Error: failed to close "testdata/logs.json.gz": close testdata/logs.json.gz: file already closed

Happens when ingestig. Mind taking a look with me what could cause this? Need a second pair of eyes @bahlo

github-actions[bot] commented 2 years ago

This PR/issue depends on: