-
Notifications
You must be signed in to change notification settings - Fork 572
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
fix the issue #188: add the file name and code line #214
Conversation
pkg/exec/cli.go
Outdated
@@ -28,7 +28,6 @@ import ( | |||
|
|||
// used in tests | |||
var disableMacOSChecks bool | |||
var disableLinuxChecks bool |
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.
why delete this?
I think this format makes the most sense because it's both * human-readable * machine-readable (this is RFC3339 with microseconds) * relatively popular * a-z sortable * less ambiguous when it comes to month/day vs day/month conventions * has good enough precision for our use-case but not too much
Codecov Report
@@ Coverage Diff @@
## main #214 +/- ##
==========================================
- Coverage 57.36% 56.90% -0.46%
==========================================
Files 73 73
Lines 2943 2988 +45
==========================================
+ Hits 1688 1700 +12
- Misses 1110 1137 +27
- Partials 145 151 +6
Continue to review full report at Codecov.
|
Add createdBy metadata to point to fire and its version
the output format is like:
INFO[25-05-2021 23:05:56] cli/cli.go:415 starting HTTP server