Skip to content

Migrate sln to slnx#57

Open
ratijas wants to merge 1 commit intoTeamXiaolan:mainfrom
ratijas:work/ratijas/slnx
Open

Migrate sln to slnx#57
ratijas wants to merge 1 commit intoTeamXiaolan:mainfrom
ratijas:work/ratijas/slnx

Conversation

@ratijas
Copy link
Contributor

@ratijas ratijas commented Jan 31, 2026

@ratijas ratijas force-pushed the work/ratijas/slnx branch 3 times, most recently from 3a8af5f to efcf105 Compare February 2, 2026 12:49
@ratijas ratijas marked this pull request as draft February 2, 2026 13:09
@ratijas
Copy link
Contributor Author

ratijas commented Feb 2, 2026

Draft because we don't know whether CI can handle this until it's time to publish a release

@ratijas ratijas marked this pull request as ready for review February 3, 2026 11:53
@XuuXiaolan
Copy link
Collaborator

I wanna shelf this for a bit to investigate a bit more, it works for me but didn't for another maintainer (they also couldn't get access to preview C# LangVersion even though I could so I had to regress that)

@ratijas
Copy link
Contributor Author

ratijas commented Feb 3, 2026

It works in CI with 8.0.x dotnet SDK. Keep in mind that after switching sln format (applying the patch) you need to run command > Developer: Reload Window in VS Code or some equivalent of reloading the project / in other IDE. Seems like C#/.NET LSP is unable to pick up such changes on the fly, so it momentarily breaks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants