-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Using %O for errors in
| log.error('Caught Error when creating context in buildVM, %O', error); |
is worse than
log.error(error, message). Not just editing it because this logs the error twice with different messages, first here and then in errorReporter.message.
- Consider adding
alreadyLoggedorlabelas parameters toerrorReporter.message/error, they are probably useful for other calls as well. - Add an ESLint rule banning
%O,%o, and%j(all are the same) by default.
Metadata
Metadata
Assignees
Labels
No labels