Skip to content

Update CD logging to include override for line prefix#351

Merged
cobya merged 2 commits into
mainfrom
cobya/LoggerVerbosityParam
Nov 7, 2022
Merged

Update CD logging to include override for line prefix#351
cobya merged 2 commits into
mainfrom
cobya/LoggerVerbosityParam

Conversation

@cobya
Copy link
Copy Markdown
Contributor

@cobya cobya commented Nov 7, 2022

Add a line prefix override for the Logger class to skip printing [INFO], [VERBOSE], etc. and update log grouping slightly.

@cobya cobya requested a review from a team as a code owner November 7, 2022 17:03
@cobya cobya requested a review from annaowens November 7, 2022 17:03
Comment thread src/Microsoft.ComponentDetection.Orchestrator/Orchestrator.cs
Copy link
Copy Markdown
Member

@JamieMagee JamieMagee left a comment

Choose a reason for hiding this comment

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

1 tiny comment

@cobya cobya merged commit 424fa3e into main Nov 7, 2022
@cobya cobya deleted the cobya/LoggerVerbosityParam branch November 7, 2022 17:12
@JamieMagee JamieMagee restored the cobya/LoggerVerbosityParam branch November 7, 2022 22:22
@JamieMagee JamieMagee deleted the cobya/LoggerVerbosityParam branch November 7, 2022 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants