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.

Sign up for GitHub

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

Jump to bottom

Added winston logging #50

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Added winston logging #50

merged 1 commit into from
Nov 5, 2024

Conversation

timwekkenbc
Copy link
Collaborator

  • Added winston logging
  • Added universal app logger
  • Updated existing logging to use this logger

@timwekkenbc timwekkenbc force-pushed the feature/winston-logging branch 5 times, most recently from e926279 to 0786361 Compare November 4, 2024 23:48
@timwekkenbc timwekkenbc force-pushed the feature/winston-logging branch from 0786361 to c389b69 Compare November 5, 2024 16:44
brysonjbest
brysonjbest approved these changes Nov 5, 2024
View reviewed changes
Copy link
Collaborator

@brysonjbest brysonjbest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really like the conciseness of the logging for development, and hopefully the Kibana compatibility will be helpful in our production logs!

@timwekkenbc timwekkenbc merged commit 405a845 into dev Nov 5, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brysonjbest brysonjbest brysonjbest approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timwekkenbc @brysonjbest