Skip to content

Fix spans for cached items at the manager level, not the service level#53261

Merged
CyrusNajmabadi merged 8 commits intodotnet:mainfrom
CyrusNajmabadi:cachedItemSpan
May 7, 2021
Merged

Fix spans for cached items at the manager level, not the service level#53261
CyrusNajmabadi merged 8 commits intodotnet:mainfrom
CyrusNajmabadi:cachedItemSpan

Conversation

@CyrusNajmabadi
Copy link
Contributor

Followup to #53259

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner May 7, 2021 18:02
@ghost ghost added the Area-IDE label May 7, 2021
@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner May 7, 2021 18:09
Copy link
Member

@genlu genlu left a comment

Choose a reason for hiding this comment

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

:shipit:

Copy link
Member

@333fred 333fred left a comment

Choose a reason for hiding this comment

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

OmniSharp change LGTM

@CyrusNajmabadi CyrusNajmabadi enabled auto-merge May 7, 2021 18:46
@333fred 333fred disabled auto-merge May 7, 2021 19:30
@333fred
Copy link
Member

333fred commented May 7, 2021

@CyrusNajmabadi sorry to disable auto-merge, but can you please take this for 16.11 or 16.10 instead?

@CyrusNajmabadi CyrusNajmabadi enabled auto-merge May 7, 2021 19:44
@CyrusNajmabadi
Copy link
Contributor Author

Still auto-merging. We can make anotehr PR to backport :)

@CyrusNajmabadi CyrusNajmabadi disabled auto-merge May 7, 2021 19:44
@CyrusNajmabadi CyrusNajmabadi enabled auto-merge May 7, 2021 19:44
@CyrusNajmabadi CyrusNajmabadi merged commit 050c263 into dotnet:main May 7, 2021
@ghost ghost added this to the Next milestone May 7, 2021
@CyrusNajmabadi CyrusNajmabadi deleted the cachedItemSpan branch May 7, 2021 20:02
@RikkiGibson RikkiGibson removed this from the Next milestone Jun 29, 2021
@RikkiGibson RikkiGibson added this to the 17.0.P2 milestone Jun 29, 2021
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.

4 participants