Skip to content

Conversation

shsuman
Copy link
Member

@shsuman shsuman commented Sep 1, 2020

For #

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
  • Title summarizes what is changing.
  • Has a news entry file (remember to thank yourself!).
  • Appropriate comments and documentation strings in the code.
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated.
  • Test plan is updated as appropriate.
  • package-lock.json has been regenerated by running npm install (if dependencies have changed).
  • The wiki is updated with any design decisions/details.

Cherry picks these commits :

8e00525: Fix: Getting an error on selecting an existing CI which has a display name from the historical CI's in the quick picks (#13642)
a5699d4: Make Jupyter Server name clickale to select Jupyter server (#13657)

@shsuman shsuman changed the base branch from master to release-2020.08 September 1, 2020 08:21
@codecov-commenter
Copy link

codecov-commenter commented Sep 1, 2020

Codecov Report

Merging #13712 into release-2020.08 will decrease coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           release-2020.08   #13712      +/-   ##
===================================================
- Coverage            59.75%   59.73%   -0.03%     
===================================================
  Files                  671      671              
  Lines                37382    37383       +1     
  Branches              5312     5313       +1     
===================================================
- Hits                 22338    22331       -7     
- Misses               13909    13914       +5     
- Partials              1135     1138       +3     
Impacted Files Coverage Δ
src/client/datascience/jupyter/jupyterUtils.ts 52.17% <0.00%> (ø)
src/client/datascience/types.ts 100.00% <ø> (ø)
.../datascience/interactive-common/interactiveBase.ts 5.68% <50.00%> (+0.15%) ⬆️
src/client/datascience/jupyter/serverSelector.ts 73.40% <50.00%> (+0.28%) ⬆️
src/client/datascience/common.ts 58.44% <100.00%> (ø)
src/client/common/utils/platform.ts 64.70% <0.00%> (-11.77%) ⬇️
src/client/datascience/crossProcessLock.ts 79.41% <0.00%> (-11.77%) ⬇️
src/client/linters/pydocstyle.ts 86.66% <0.00%> (-2.23%) ⬇️
src/client/datascience/debugLocationTracker.ts 76.56% <0.00%> (-1.57%) ⬇️
src/client/common/process/proc.ts 14.49% <0.00%> (-0.73%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c9d5bd...9f86a92. Read the comment docs.

@brettcannon
Copy link
Member

Was there a specific reason I was asked to review this?

@rchiodo
Copy link

rchiodo commented Sep 1, 2020

Was there a specific reason I was asked to review this?

No I believe you were just on the recommended list.

@brettcannon brettcannon removed their request for review September 1, 2020 16:56
@shsuman
Copy link
Member Author

shsuman commented Sep 1, 2020

Was there a specific reason I was asked to review this?

No

@shsuman shsuman added the no-changelog No news entry required label Sep 1, 2020
@shsuman shsuman force-pushed the shsuman/cherry_picks_for_point_release branch from f82e540 to 9f86a92 Compare September 1, 2020 18:07
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 1, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@shsuman shsuman merged commit af35cb6 into microsoft:release-2020.08 Sep 1, 2020
karthiknadig added a commit that referenced this pull request Sep 8, 2020
* Port gather survey and icon getting disabled while gathering (#13296)

* Add gather status bar and update survey link (#13225)

* add status bar

* add localization

* Added news file

* add a 2 second timeout

* updated the gather survey link

* Change the gather icon while gathering

* moved the gathering redux function to
interactive common
added a constant

* move css to common.css

* removed the status bar message

* changed noop import

* update changelog

* Disable test (#13346) (#13350)

Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Add cell navigation commands as well as a default set of keyboard bindings for most new interactive window cell commands (#13334) (#13348)

* Add Goto Prev/Next Cell

* Add goto previous cell

* Remove unneeded changes

* Add default keybindings for cell commands

* Goto should be Go to

Co-authored-by: Jim Griesmer <jimg@live.com>

Co-authored-by: Jim Griesmer <jimg@live.com>

* Autoscroll when extending cell select (#13353) (#13354)

Code change can only affect new keyboard shortcut operations

* Update Release branch for August Release (#13401)

* update version number and npm install to update package-lock.json

* update changelog version and date

* update news for changelog.md

* update generated TPN

Co-authored-by: Ian Huff <ianhuff@ravikun-dev2.redmond.corp.microsoft.com>

* Update version and change log for bug fix release (#13537)

* Update LSP to latest to resolve problems with LS settings  (#13511)

* Fix path

* Actually fix settings

* Add news

* Add test

* Format

* Suppress 'jediEnabled' removal

* Drop survey first launch threshold

* Update LSP to fix issue with settings in MPLS v1

* Add telemetry for `switch to Pylance` acceptance (#13526)

* Fix path

* Actually fix settings

* Add news

* Add test

* Format

* Suppress 'jediEnabled' removal

* Drop survey first launch threshold

* Initial

* Add test

* PR feedback

* Rename property

* Rename constant

* Define property

* Moar rename

* casing

* Update version and changelogs

* Show status of Jupyter server (Connected/Disconnected) when the user hovers over the status icon (#13544)

* Update change log with more cherry picks

Co-authored-by: Mikhail Arkhipov <mikhaila@microsoft.com>
Co-authored-by: Shantnu Suman <shsuman@microsoft.com>

* Update changelog and version for point release. (#13610)

* Show the server display string that the user is going to connect to after selecting a compute instance and reloading the window. (#13600)

* Update "Tip" notification for new users to either show the existing tip, a link to a feedback survey or nothing (#13554)

* Update vscode-tas-client
* Add experiment group enum
* Add method to retrieve experiment values
* Implementation + tests
* News file
* Update wording of the news entry
* Add telemetry
* More tests
* No opting-in and out of this one
* Don't fetch value if opted out, add tests
* Address comments

* Fix save on close (#13567)

* Pass model through command instead of URI in order to use directly

* Add a test to verify we don't regress

* Finish fixing tests

* Add news entry

* Fix unit tests

* Update changelog and version for point release.

* Change date.

* Format using latest black (#13625)

Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Skip flaky virtual env tests (#13650)

* Show the server display string that the user is going to connect to after selecting a compute instance and reloading the window. (#13600)

* Update "Tip" notification for new users to either show the existing tip, a link to a feedback survey or nothing (#13554)

* Update vscode-tas-client
* Add experiment group enum
* Add method to retrieve experiment values
* Implementation + tests
* News file
* Update wording of the news entry
* Add telemetry
* More tests
* No opting-in and out of this one
* Don't fetch value if opted out, add tests
* Address comments

* Fix save on close (#13567)

* Pass model through command instead of URI in order to use directly

* Add a test to verify we don't regress

* Finish fixing tests

* Add news entry

* Fix unit tests

* Update changelog and version for point release.

* Skip workspace virtual env service tests

* Add link to issue for skipping

Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Cherry picks, change log, and version update for point release. (#13666)

* Add install telemetry (#13653)

* Add extension install telemetry.

* Add tests.

* Fix typo

* Fix tests.

* Update comment to reflect what the installed actually does.

* Update change log and version for point release.

* Cherry pick history bug for Azure ML CI && Make Jupyter server clickable (#13712)

* Fix: Getting an error on selecting an existing CI which has a display name from the historical CI's in the quick picks (#13642)

* Make Jupyter Server name clickale to select Jupyter server (#13657)

* Update Changelog and remove news files

* Update package version. (#13748)

* Update and pin LSP libraries for new APIs (#13725) (#13756)

Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>

* PORT showStartPage setting fix (#13706) (#13758)

* fix showStartPage setting on configSettings.ts (#13706)

* fix showStartPage setting on configSettings.ts

* add news file

* Fix unit tests

* oops

* update changelog

* Fix merge issues

* Clean up news

Co-authored-by: David Kutugata <dakutuga@microsoft.com>
Co-authored-by: Ian Huff <ianhu@microsoft.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>
Co-authored-by: Jim Griesmer <jimgries@microsoft.com>
Co-authored-by: Jim Griesmer <jimg@live.com>
Co-authored-by: Ian Huff <ianhuff@ravikun-dev2.redmond.corp.microsoft.com>
Co-authored-by: Mikhail Arkhipov <mikhaila@microsoft.com>
Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
karthiknadig added a commit that referenced this pull request Sep 14, 2020
* Port gather survey and icon getting disabled while gathering (#13296)

* Add gather status bar and update survey link (#13225)

* add status bar

* add localization

* Added news file

* add a 2 second timeout

* updated the gather survey link

* Change the gather icon while gathering

* moved the gathering redux function to
interactive common
added a constant

* move css to common.css

* removed the status bar message

* changed noop import

* update changelog

* Disable test (#13346) (#13350)

Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Add cell navigation commands as well as a default set of keyboard bindings for most new interactive window cell commands (#13334) (#13348)

* Add Goto Prev/Next Cell

* Add goto previous cell

* Remove unneeded changes

* Add default keybindings for cell commands

* Goto should be Go to

Co-authored-by: Jim Griesmer <jimg@live.com>

Co-authored-by: Jim Griesmer <jimg@live.com>

* Autoscroll when extending cell select (#13353) (#13354)

Code change can only affect new keyboard shortcut operations

* Update Release branch for August Release (#13401)

* update version number and npm install to update package-lock.json

* update changelog version and date

* update news for changelog.md

* update generated TPN

Co-authored-by: Ian Huff <ianhuff@ravikun-dev2.redmond.corp.microsoft.com>

* Update version and change log for bug fix release (#13537)

* Update LSP to latest to resolve problems with LS settings  (#13511)

* Fix path

* Actually fix settings

* Add news

* Add test

* Format

* Suppress 'jediEnabled' removal

* Drop survey first launch threshold

* Update LSP to fix issue with settings in MPLS v1

* Add telemetry for `switch to Pylance` acceptance (#13526)

* Fix path

* Actually fix settings

* Add news

* Add test

* Format

* Suppress 'jediEnabled' removal

* Drop survey first launch threshold

* Initial

* Add test

* PR feedback

* Rename property

* Rename constant

* Define property

* Moar rename

* casing

* Update version and changelogs

* Show status of Jupyter server (Connected/Disconnected) when the user hovers over the status icon (#13544)

* Update change log with more cherry picks

Co-authored-by: Mikhail Arkhipov <mikhaila@microsoft.com>
Co-authored-by: Shantnu Suman <shsuman@microsoft.com>

* Update changelog and version for point release. (#13610)

* Show the server display string that the user is going to connect to after selecting a compute instance and reloading the window. (#13600)

* Update "Tip" notification for new users to either show the existing tip, a link to a feedback survey or nothing (#13554)

* Update vscode-tas-client
* Add experiment group enum
* Add method to retrieve experiment values
* Implementation + tests
* News file
* Update wording of the news entry
* Add telemetry
* More tests
* No opting-in and out of this one
* Don't fetch value if opted out, add tests
* Address comments

* Fix save on close (#13567)

* Pass model through command instead of URI in order to use directly

* Add a test to verify we don't regress

* Finish fixing tests

* Add news entry

* Fix unit tests

* Update changelog and version for point release.

* Change date.

* Format using latest black (#13625)

Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Skip flaky virtual env tests (#13650)

* Show the server display string that the user is going to connect to after selecting a compute instance and reloading the window. (#13600)

* Update "Tip" notification for new users to either show the existing tip, a link to a feedback survey or nothing (#13554)

* Update vscode-tas-client
* Add experiment group enum
* Add method to retrieve experiment values
* Implementation + tests
* News file
* Update wording of the news entry
* Add telemetry
* More tests
* No opting-in and out of this one
* Don't fetch value if opted out, add tests
* Address comments

* Fix save on close (#13567)

* Pass model through command instead of URI in order to use directly

* Add a test to verify we don't regress

* Finish fixing tests

* Add news entry

* Fix unit tests

* Update changelog and version for point release.

* Skip workspace virtual env service tests

* Add link to issue for skipping

Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>

* Cherry picks, change log, and version update for point release. (#13666)

* Add install telemetry (#13653)

* Add extension install telemetry.

* Add tests.

* Fix typo

* Fix tests.

* Update comment to reflect what the installed actually does.

* Update change log and version for point release.

* Cherry pick history bug for Azure ML CI && Make Jupyter server clickable (#13712)

* Fix: Getting an error on selecting an existing CI which has a display name from the historical CI's in the quick picks (#13642)

* Make Jupyter Server name clickale to select Jupyter server (#13657)

* Update Changelog and remove news files

* Update package version. (#13748)

* Update and pin LSP libraries for new APIs (#13725) (#13756)

Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>

* PORT showStartPage setting fix (#13706) (#13758)

* fix showStartPage setting on configSettings.ts (#13706)

* fix showStartPage setting on configSettings.ts

* add news file

* Fix unit tests

* oops

* update changelog

* Fixes for point release (#13841)

* Use 'main' branch for experiments.

* Update changelog and versions

* Update CHANGELOG.md

* Replace 'master' with 'main' in readme (#13845)

Co-authored-by: David Kutugata <dakutuga@microsoft.com>
Co-authored-by: Ian Huff <ianhu@microsoft.com>
Co-authored-by: Rich Chiodo <rchiodo@users.noreply.github.com>
Co-authored-by: Jim Griesmer <jimgries@microsoft.com>
Co-authored-by: Jim Griesmer <jimg@live.com>
Co-authored-by: Ian Huff <ianhuff@ravikun-dev2.redmond.corp.microsoft.com>
Co-authored-by: Mikhail Arkhipov <mikhaila@microsoft.com>
Co-authored-by: Shantnu Suman <shsuman@microsoft.com>
Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>
Co-authored-by: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
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.

6 participants