New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Snapshots #83
Conversation
✅ Deploy Preview for raredocs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov Report
@@ Coverage Diff @@
## master #83 +/- ##
==========================================
+ Coverage 86.56% 87.12% +0.56%
==========================================
Files 93 96 +3
Lines 4363 4390 +27
==========================================
+ Hits 3777 3825 +48
+ Misses 450 431 -19
+ Partials 136 134 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@@ -59,8 +59,7 @@ func analyzeFunction(c *cli.Context) error { | |||
} | |||
|
|||
aggr := aggregation.NewNumericalAggregator(&config) | |||
writer := multiterm.New() | |||
defer multiterm.ResetCursor() | |||
writer := helpers.BuildVTermFromArguments(c) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this defer writer.close (and all other agg's?) Is it safer?
Add ability to "snapshot" output for out pipes and file writes