You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Intent of error handling unclear. If you just want to log the error and continue, empty else blocks are confusing.
Also better to write a custom log function for two reasons, first so that you can add extra information to the log message in a structured way, and second so the reader can infer that the logging is intentional and not just left-over from debugging:
Intent of error handling unclear. If you just want to log the error and continue, empty
else
blocks are confusing.Also better to write a custom log function for two reasons, first so that you can add extra information to the log message in a structured way, and second so the reader can infer that the logging is intentional and not just left-over from debugging:
https://github.com/FAC10/MPBots/blob/master/src/helper_functions/checkAPIAI.js#L39-L42
The text was updated successfully, but these errors were encountered: