-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
@droberts-ctrlo
droberts-ctrlo
left a comment
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.
Reviewed - no apparent issues
@pwlodarski-ctrlo
pwlodarski-ctrlo
left a comment
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.
PR reviewed - except a minor comment, all is well.
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.
Debug statement?
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 Pawel and good spot but this is actually a "proper" way of doing debugging in the code - these trace statements will only print anything when the log level is high enough.
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.
Understood, thanks for clarifying Andy!
No description provided.