MinaFoundation / gptSurveySummarizer

Discord bot to gather your thoughts
2 stars 2 forks source link

Add LOGLEVEL with Timestamps to logs #48

Closed berkingurcan closed 2 weeks ago

berkingurcan commented 4 weeks ago

As @johnmarcou requested, it is good practice to add log level for logs with timestamps. Can be used: https://www.npmjs.com/package/loglevel

berkingurcan commented 2 weeks ago

Solved: https://github.com/MinaFoundation/gptSurveySummarizer/pull/56 But need to check how to configure in ENV, because I set the log level in here:

https://github.com/MinaFoundation/gptSurveySummarizer/pull/56/commits/89d4b76b7f4c291bf306e2c840a176c1feffd5fb#diff-4c57e1eda9c7d9cb0e46d669ec83dfb50a214e0263ddef7b2245d5a5bc4d57d9

Where should I get the LOG LEVEL value? @johnmarcou

berkingurcan commented 2 weeks ago
Screenshot 2024-06-10 at 15 25 02
johnmarcou commented 2 weeks ago

Solved: #56 But need to check how to configure in ENV, because I set the log level in here:

89d4b76#diff-4c57e1eda9c7d9cb0e46d669ec83dfb50a214e0263ddef7b2245d5a5bc4d57d9

Where should I get the LOG LEVEL value? @johnmarcou

@berkingurcan Usually it's nice to have an environment variable to set the logging level, such as GPT_SURVEY_SUMMARIZER_LOG_LEVEL, or may be just GSS_LOG_LEVEL. Default could be info.