-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add logging to the app #11
Comments
This could be covered by these two solutions. |
I think best option is to use console logging no need for any additional dependencies for the project of this size. |
It would mean less dependencies, but wouldn't provide f.e. severity of the log entry and would potentialy create a tech debt with the project size increasing in the future. |
I don't think this project can get to the size where using simpler logging tools would create a tech debt (mind it contains other methods such as console.warn, console.error so it already has basic tools for logging). On the other hand it will ensure that if this project would get into the maintenance mode there would be one less library we need to keep updated... |
It would be good for future debugging to add logging to the app.
Acceptance criteria
The text was updated successfully, but these errors were encountered: