-
Notifications
You must be signed in to change notification settings - Fork 70
Closed
Labels
good first issueGood for newcomersGood for newcomersmodule/libRelated to the main source codeRelated to the main source codestatus/no-issue-activity
Description
This is not really a bug, nor a feature request, so I haven't followed any of the templates.
It seems console.error
is being used in several situations, e.g.:
sdk-javascript/src/event/cloudevent.ts
Line 170 in 6cd310c
console.error(e.errors); |
In most scenarios, these console.error
calls are unnecessary as errors will be thrown anyway, that should be handled by devs consuming this library. In other situations, errors aren't thrown, and these messages are useful. However, most devs are using custom loggers (like JSON loggers), and would like to be able to customize the logging behavior.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersmodule/libRelated to the main source codeRelated to the main source codestatus/no-issue-activity