Skip to content

Move some nuget-search code to run in the BG#59540

Merged
CyrusNajmabadi merged 7 commits intodotnet:mainfrom
CyrusNajmabadi:bgLoads
Feb 14, 2022
Merged

Move some nuget-search code to run in the BG#59540
CyrusNajmabadi merged 7 commits intodotnet:mainfrom
CyrusNajmabadi:bgLoads

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

@CyrusNajmabadi CyrusNajmabadi commented Feb 14, 2022

Fixes AB#1281349

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner February 14, 2022 20:00
@ghost ghost added the Area-IDE label Feb 14, 2022
packageSourceProvider.SourcesChanged += OnSourceProviderSourcesChanged;
}

protected override void StartWorking()
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

StartWorking got merged into EnableServiceAsync since we only called both at teh same time, one after the other.

Copy link
Copy Markdown
Member

@jasonmalinowski jasonmalinowski left a comment

Choose a reason for hiding this comment

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

Some small comments you'll want to fix, but otherwise :shipit:.

@CyrusNajmabadi CyrusNajmabadi merged commit c446c82 into dotnet:main Feb 14, 2022
@ghost ghost added this to the Next milestone Feb 14, 2022
@CyrusNajmabadi CyrusNajmabadi deleted the bgLoads branch February 18, 2022 20:49
@RikkiGibson RikkiGibson modified the milestones: Next, 17.2.P2 Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants