Skip to content

Conversation

@jasonmalinowski
Copy link
Member

This gets us loading on 17.14 builds again.

@jasonmalinowski jasonmalinowski self-assigned this Mar 6, 2025
@jasonmalinowski jasonmalinowski requested a review from a team as a code owner March 6, 2025 02:31
@ghost ghost added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Mar 6, 2025
@jasonmalinowski jasonmalinowski removed the untriaged Issues and PRs which have not yet been triaged by a lead label Mar 7, 2025
@jasonmalinowski
Copy link
Member Author

Merging as this means we go from "entirely unable to load Roslyn in integration tests" to "just CodeLens seems broken". That's not green, but a significant improvement.

@jasonmalinowski jasonmalinowski merged commit 2f8e4d0 into dotnet:main-vs-deps Mar 7, 2025
21 of 25 checks passed
@jasonmalinowski jasonmalinowski deleted the fix-mef-loading-against-17.14 branch March 7, 2025 23:00
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Mar 7, 2025
@jasonmalinowski
Copy link
Member Author

Overriding the merge while we sort out whether we should still be requiring integration tests or not.

SPGLP added a commit to SPGLP/roslyn that referenced this pull request Mar 12, 2025
jasonmalinowski added a commit that referenced this pull request Mar 24, 2025
We recently made some [changes to upgrade to the 9.0 versions of various
.NET packages](#76890); this had
the effect of breaking our integration tests since the product wouldn't
run at all on older versions of VS. I made a fix [for that
problem](#77451) but integration
tests were still failing. CodeLens is legitimately broken in this case
right now, but this skips failing integration tests for that reason,
since the tests otherwise pass fine.
@jjonescz jjonescz modified the milestones: Next, 17.14 P3 Apr 1, 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