Skip to content

Conversation

@juliojordan
Copy link

@juliojordan juliojordan commented Nov 9, 2025

What does this PR do? What is the motivation?

Running the current example code yields this output.

{"level":"info","message":"Hello simple log!"}
{"color":"blue","level":"info","message":"Hello log with metas"}

By applying this change, the output will include the timestamp as expected.

Check the content of the <FILE_NAME>.log file to confirm that Winston is logging in JSON:

{"level":"info","message":"Hello simple log!","timestamp":"2015-04-23T16:52:05.337Z"}
{"color":"blue","level":"info","message":"Hello log with metas","timestamp":"2015-04-23T16:52:05.339Z"}

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

@juliojordan juliojordan requested a review from a team as a code owner November 9, 2025 08:21
@maycmlee maycmlee self-assigned this Nov 10, 2025
@maycmlee
Copy link
Contributor

Thanks @juliojordan for opening this PR! We'll take a look and be back in touch.

@maycmlee maycmlee removed their assignment Nov 10, 2025
@maycmlee maycmlee added the under_review Sent to someone outside of the docs team for review. label Nov 10, 2025
@estherk15 estherk15 self-assigned this Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

under_review Sent to someone outside of the docs team for review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants