Skip to content

Remove prompt to install a linter - #15646

Merged
Kim-Adeline Miguel (kimadeline) merged 6 commits into
microsoft:mainfrom
kimadeline:15465-drop-pylint-notification
Mar 18, 2021
Merged

Kim-Adeline Miguel (kimadeline) merged 6 commits into
microsoft:mainfrom
kimadeline:15465-drop-pylint-notification

Conversation

@kimadeline

@kimadeline Kim-Adeline Miguel (kimadeline) commented Mar 11, 2021 •

Copy link
Copy Markdown

Closes #15465

Undoing what was done in #14731 and more

@codecov-io

Codecov (codecov-io) commented Mar 12, 2021 •

Copy link
Copy Markdown

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%) ⬇️

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