feat(logging): add timestamp

[wuruochong]

Description

Added time to end of each line. Helps identify if program hangs.

Testing

Ran program with both out of stock and fake in stock messages. Verified that time is at the end of each line.

[vp-regular]

A quick google suggests you can just change the Logger itself instead. It shouldn't be the responsibility of lookup to keep track of the time in my opinion.

https://stackoverflow.com/questions/10271373/how-can-i-add-timestamp-to-logs-using-node-js-library-winston

[wuruochong]

A quick google suggests you can just change the Logger itself instead. It shouldn't be the responsibility of lookup to keep track of the time in my opinion.

https://stackoverflow.com/questions/10271373/how-can-i-add-timestamp-to-logs-using-node-js-library-winston

Good point, moved it to logger.

[admon84]

@wuruochong I tried making a similar change with Logger native format.timestamp() but I couldn't get passed a lint error with the template literal: https://github.com/jef/nvidia-snatcher/pull/51/files

Either way, I like this format having the timestamp consistently on the front of the line, like you have it in your latest commit.

[jef]

@wuruochong I tried making a similar change with Logger native format.timestamp() but I couldn't get passed a lint error with the template literal: #51 (files)

Either way, I like this format having the timestamp consistently on the front of the line, like you have it in your latest commit.

I too was doing this originally, but decided just to leave it out without hackin' up the timestamp type. I think at this point we definitely need timestamps.

[jef]

Thanks for your contribution. Looks great 🚀 😄