-
Notifications
You must be signed in to change notification settings - Fork 79
Conversation
22c21d6
to
027c3b5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @ricardozanini . Since v0.10.0 supports configuring timestamp format for logging, I think we should make the timestamp more readable.
Hmm... the logs showed with the |
027c3b5
to
667cd11
Compare
@xiezhang7 I added the iso format to our inner log, but this change on 0.10.0 it's not meant to be used outside their CLI, as far as I understood that PR you mentioned. Anyway, now the log is much more readable:
Could you please review? |
Thanks, @ricardozanini . It looks great. |
See: https://issues.jboss.org/browse/KOGITO-271
In this PR:
Many thanks for submiting your Pull Request ❤️!
Please make sure that your PR meets the following requirements:
[KOGITO-XYZ] Subject