-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Adds logging support for python debug adapter #8272
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8272 +/- ##
==========================================
+ Coverage 59.33% 59.38% +0.05%
==========================================
Files 504 505 +1
Lines 23096 23126 +30
Branches 3737 3740 +3
==========================================
+ Hits 13704 13734 +30
Misses 8500 8500
Partials 892 892
Continue to review full report at Codecov.
|
@DonJayamanne I need some advise on testing |
e7e6b80
to
4942ecb
Compare
4942ecb
to
273654d
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.
Some typescript suggestions but other than it looks good to me 👍
c7ed650
to
e27cde8
Compare
e27cde8
to
75e4b23
Compare
For #8106
@kimadeline I need some advise on testing the logger.