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

Ignore workspaces that do not have IEditAndContinueWorkspaceService #73861

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

tmat
Copy link
Member

@tmat tmat commented Jun 5, 2024

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2048920

Workspaces that do not include Features layer wouldn't have the service (not sure what those are though).

@tmat tmat requested a review from a team as a code owner June 5, 2024 18:56
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Interactive untriaged Issues and PRs which have not yet been triaged by a lead labels Jun 5, 2024
@tmat
Copy link
Member Author

tmat commented Jun 5, 2024

@dibarbet @jasonmalinowski Any idea what workspace doesn't have features layer?

@jasonmalinowski
Copy link
Member

@tmat: oh strange; the only workspaces I can imagine would be somebody who created their own workspaces with the Default MEF services...but then how those ended up getting in a place where ENC is trying to operate doesn't make sense to me. If you want to request cabs on this one that might be wise; maybe this is really a setup issue where assemblies didn't get loaded?

@ToddGrun
Copy link
Contributor

ToddGrun commented Sep 4, 2024

Approving after chatting with Tomas. He's requested cabs, but we'd like to avoid this from getting hit in new builds.

Copy link
Contributor

@ToddGrun ToddGrun left a comment

Choose a reason for hiding this comment

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

:shipit:

@tmat tmat merged commit 7831d50 into dotnet:main Sep 4, 2024
25 checks passed
@tmat
Copy link
Member Author

tmat commented Sep 4, 2024

Will follow up on the memory dump, once we get one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Interactive untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants