Skip to content
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 LOGLEVEL with Timestamps to logs #48

Closed
berkingurcan opened this issue May 28, 2024 · 3 comments · Fixed by #56
Closed

Add LOGLEVEL with Timestamps to logs #48

berkingurcan opened this issue May 28, 2024 · 3 comments · Fixed by #56
Assignees
Labels
approved-for-product enhancement New feature or request PR 56 Survey Bot A bot to summarize and categorize surveys

Comments

@berkingurcan
Copy link
Collaborator

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

Screenshot 2024-06-10 at 15 25 02

@johnmarcou
Copy link
Collaborator

johnmarcou commented Jun 10, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-for-product enhancement New feature or request PR 56 Survey Bot A bot to summarize and categorize surveys
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants