Skip to content

Conversation

@karthiknadig
Copy link
Member

No description provided.

karthiknadig and others added 2 commits October 7, 2021 20:28
@karthiknadig karthiknadig added the skip tests Updates to tests unnecessary label Oct 8, 2021
Copy link

@karrtikr karrtikr left a comment

Choose a reason for hiding this comment

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

FYI I plan to merge #17658 EOD today.

@karthiknadig
Copy link
Member Author

@karrtikr I will pull that in as well.

@karrtikr
Copy link

karrtikr commented Oct 11, 2021

Updated list of items from my end to cherry pick for point release: #17698 #17658 #17670 #17697 #17711 #17714

Kartik Raj added 5 commits October 11, 2021 14:40
…crosoft#17670)

* Log commands run by the discovery component in the output channel

* Remove outdated comments
…crosoft#17697)

* Ensure commands run are not logged twice in Python output channel

* News entry
…rosoft#17698)

* Hide UI elements that are not applicable when using VS Code Web

* Fix indentaiton

* News entry

* Disable welcome view for testing
* Change localization in the extension to be async and use the VS Code APIs

* News entry

* Modify error thrown

* Move localization into separate module

* Update news entry

* Oops

* Refactor so code is not duplicated

* Fix tests

* Oopsp

* Fix lint
…osoft#17658)

* Provide IntelliSense status information when using github.dev

* News entry

* Update wording

* PR reviews

* Include proposed APIs for browser config

* Include common module for browser config

* Change how we detect

* Change wording

* Have same browser config as extension config

* Revert "Have same browser config as extension config"

This reverts commit 515003a.

* Revert "Change wording"

This reverts commit dd64f24.

* Revert "Change how we detect"

This reverts commit ae56079.

* Revert "Include common module for browser config"

This reverts commit bf1815c.

* Do not import from misc module

* Detect how we check if property exists

* Do not localize strings

* Localize strings using new localize module

* Remove outdated comment
@karrtikr
Copy link

#17714

@karthiknadig karthiknadig added skip package*.json package.json and package-lock.json don't both need updating and removed skip tests Updates to tests unnecessary labels Oct 13, 2021
@karthiknadig karthiknadig merged commit 9128f87 into microsoft:release-2021.10 Oct 13, 2021
@karthiknadig karthiknadig deleted the cherry-picks branch October 13, 2021 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip package*.json package.json and package-lock.json don't both need updating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants