Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass pre-release info per installed extension to get the latest such extension from marketplace #139017

Merged
merged 1 commit into from Dec 13, 2021

Conversation

sandy081
Copy link
Member

This PR fixes #139013

Pass pre-release info per installed extension to get the latest such extension from marketplace. This reduces additional requests to Marketplace for latest pre-release or release version in VS Code stable or VS Code insiders respectively.

@sandy081 sandy081 added this to the November 2021 Recovery 2 milestone Dec 13, 2021
@sandy081 sandy081 self-assigned this Dec 13, 2021
@isidorn
Copy link
Contributor

isidorn commented Dec 13, 2021

Change looks good, introduced a new method which is only used in this case and I have verified that the MP request count drops as expected -> approved.

@sandy081 sandy081 merged commit e7a2ed3 into release/1.63 Dec 13, 2021
@sandy081 sandy081 deleted the sandy081/fix139013 branch December 13, 2021 21:00
deepak1556 added a commit that referenced this pull request Dec 14, 2021
deepak1556 added a commit that referenced this pull request Dec 14, 2021
* Revert "re-enable smoke tests, keep editor fix (#139012) (#139019)"

This reverts commit c4738fe.

* Revert "Add yarn file (#139035)"

This reverts commit b3d3156.

* Revert "Bump distro and version for 1.63.1 (#139034)"

This reverts commit bc22b67.

* Revert "Pick up TS recovery (#138795)"

This reverts commit 223411f.

* Revert "Merge pull request #139017 from microsoft/sandy081/fix139013"

This reverts commit e7a2ed3, reversing
changes made to 7db1a2b.
@sandy081 sandy081 restored the sandy081/fix139013 branch December 15, 2021 08:04
@github-actions github-actions bot locked and limited conversation to collaborators Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants