Skip to content

Conversation

@CyrusNajmabadi
Copy link
Member

Followup to #80321.

@CyrusNajmabadi CyrusNajmabadi marked this pull request as ready for review September 17, 2025 12:24
@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner September 17, 2025 12:24
@CyrusNajmabadi
Copy link
Member Author

@dotnet/roslyn-ide this is ready for review

@CyrusNajmabadi CyrusNajmabadi enabled auto-merge (rebase) September 17, 2025 12:31
auto-merge was automatically disabled September 17, 2025 13:56

Rebase failed

@CyrusNajmabadi CyrusNajmabadi enabled auto-merge (rebase) September 17, 2025 14:03
auto-merge was automatically disabled September 17, 2025 14:56

Rebase failed

@CyrusNajmabadi CyrusNajmabadi merged commit 1a8bea9 into dotnet:main Sep 17, 2025
25 checks passed
@CyrusNajmabadi CyrusNajmabadi deleted the sigHelpImmutable branch September 17, 2025 14:58
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Sep 17, 2025
@akhera99 akhera99 modified the milestones: Next, 18.0 P1, 18.0 P2 Sep 22, 2025
CyrusNajmabadi added a commit to CyrusNajmabadi/roslyn that referenced this pull request Sep 23, 2025
CyrusNajmabadi added a commit to CyrusNajmabadi/roslyn that referenced this pull request Sep 23, 2025
akhera99 pushed a commit that referenced this pull request Sep 23, 2025
Right now we have an issue where users are getting broken MEF
compositions cached on their machines, so subsequent launches of the
product are operating on a broken composition. It's not "broken"
in the sense that "the cache cannot be loaded" but rather the content
of the cache is just missing most of our MEF parts. As a defense-in-
depth change to deal with potential transient issues, let's not cache
compositions we know are bad.

Co-authored-by: Jason Malinowski <[email protected]>
akhera99 pushed a commit that referenced this pull request Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants