Use format string in log.Fatalf for error messages
- Changed
log.Fatalf(err.Error())tolog.Fatalf("%s", err.Error())
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mahmut-Abi Once this PR has been reviewed and has the lgtm label, please assign tessaio for approval. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
log.Fatalf ( “% s ” , err.Error ( ) ) It is better to replace with "log.Fatal". Or complete the newline character "%s\n".
Sorry for the long day. I agree with @LJTian
Closing for now! Thanks for the PR.