Skip to content

Fix typo in the word selection - #15309

Merged
Karthik Nadig (karthiknadig) merged 2 commits into
microsoft:mainfrom
karthiknadig:typo1
Feb 11, 2021
Merged

Fix typo in the word selection#15309
Karthik Nadig (karthiknadig) merged 2 commits into
microsoft:mainfrom
karthiknadig:typo1

Conversation

@karthiknadig

Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me. I didn't check, did you do a linting pass?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@karthiknadig

Copy link
Copy Markdown
Member Author

Pending linting pass

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #15309 (44abf5c) into main (790c909) will increase coverage by 0%.
The diff coverage is 74%.

@@           Coverage Diff           @@
##            main   #15309    +/-   ##
=======================================
  Coverage     65%      65%            
=======================================
  Files        558      559     +1     
  Lines      26646    26799   +153     
  Branches    3840     3868    +28     
=======================================
+ Hits       17322    17447   +125     
- Misses      8617     8634    +17     
- Partials     707      718    +11     
Impacted Files Coverage Δ
src/client/activation/jedi/analysisOptions.ts 55% <0%> (-16%) ⬇️
...rc/client/activation/jedi/languageClientFactory.ts 46% <0%> (+6%) ⬆️
src/client/activation/jedi/manager.ts 29% <ø> (ø)
src/client/common/application/commandManager.ts 25% <0%> (ø)
src/client/common/application/types.ts 100% <ø> (ø)
src/client/common/installer/productInstaller.ts 40% <ø> (ø)
src/client/common/platform/types.ts 100% <ø> (ø)
src/client/common/terminal/types.ts 100% <ø> (ø)
src/client/interpreter/interpreterService.ts 60% <0%> (ø)
src/client/providers/completionProvider.ts 30% <0%> (-3%) ⬇️
... and 51 more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants