Skip to content

Conversation

kimadeline
Copy link

@kimadeline kimadeline commented Mar 11, 2021

Closes #15465

Undoing what was done in #14731 and more

@kimadeline kimadeline self-assigned this Mar 11, 2021
@codecov-io
Copy link

codecov-io commented Mar 12, 2021

Codecov Report

Merging #15646 (947a46a) into main (a1911a5) will decrease coverage by 0%.
The diff coverage is 100%.

@@           Coverage Diff           @@
##            main   #15646    +/-   ##
=======================================
- Coverage     64%      64%    -1%     
=======================================
  Files        557      557            
  Lines      26813    26700   -113     
  Branches    3901     3879    -22     
=======================================
- Hits       17369    17280    -89     
+ Misses      8710     8695    -15     
+ Partials     734      725     -9     
Impacted Files Coverage Δ
src/client/common/experiments/groups.ts 100% <ø> (ø)
src/client/telemetry/constants.ts 100% <ø> (ø)
src/client/telemetry/index.ts 77% <ø> (ø)
src/client/common/installer/productInstaller.ts 26% <100%> (-16%) ⬇️

@kimadeline kimadeline closed this Mar 12, 2021
@kimadeline kimadeline reopened this Mar 12, 2021
@kimadeline kimadeline marked this pull request as ready for review March 15, 2021 21:12
@kimadeline kimadeline marked this pull request as draft March 17, 2021 16:30
@kimadeline kimadeline marked this pull request as ready for review March 17, 2021 16:30
@kimadeline kimadeline merged commit f214544 into microsoft:main Mar 18, 2021
@kimadeline kimadeline deleted the 15465-drop-pylint-notification branch March 18, 2021 19:29
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.

Drop pylint notification experiment

4 participants