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.

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

hard code diagnostic logging #75

Merged
merged 1 commit into from
May 6, 2024
Merged

hard code diagnostic logging #75

merged 1 commit into from
May 6, 2024

Conversation

tonybaloney
Copy link
Contributor

Purpose

  • ...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@tonybaloney tonybaloney merged commit a8bbfb5 into main May 6, 2024
4 checks passed
@tonybaloney tonybaloney deleted the diagnostics2 branch May 6, 2024 02:24
detailedErrorLoggingEnabled: enableDiagnosticLogging
httpLoggingEnabled: enableDiagnosticLogging
requestTracingEnabled: enableDiagnosticLogging
detailedErrorLoggingEnabled: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, will you be suggesting this change for azure-dev? Or it was needed here for some reason, you couldnt pass in true? Ah you're trying to make template-analyzer happy, I guess?

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