OqixDevs / OqixTS

Discord BOT intended for FIT@FI server.
MIT License
4 stars 3 forks source link

Add logging to the app #11

Open mimotej opened 2 years ago

mimotej commented 2 years ago

It would be good for future debugging to add logging to the app.

Acceptance criteria

tkosci commented 1 year ago

This could be covered by these two solutions. Most used dedicated packages for logging are apparently typescript-logging and tslog. If we ignore javascript modules and plain console logging, the latter option would be propably better, since it seems easier to set up and more feature rich.

mimotej commented 1 year ago

This could be covered by these two solutions. Most used dedicated packages for logging are apparently typescript-logging and tslog. If we ignore javascript modules and plain console logging, the latter option would be propably better, since it seems easier to set up and more feature rich.

I think best option is to use console logging no need for any additional dependencies for the project of this size.

tkosci commented 1 year ago

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.

mimotej commented 1 year ago

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...