Skip to content

Conversation

@thegu5
Copy link
Contributor

@thegu5 thegu5 commented Nov 1, 2024

Both VS and (recently) Rider support the format.

If this is merged now, the .sln file will still have to be relied upon and kept in sync if any changes are made to the .slnx file, since the new format isn't supported everywhere yet. The biggest example is the dotnet CLI, which has release of the feature planned for 9.0 (dotnet/sdk#40913). Rider has a feature to convert between the two.

(Due to the above, maybe hold off on merging?)

🎃

@coveralls
Copy link

coveralls commented Nov 1, 2024

Pull Request Test Coverage Report for Build 19786088216

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 34.31%

Totals Coverage Status
Change from base Build 18333364266: 0.0%
Covered Lines: 4191
Relevant Lines: 10847

💛 - Coveralls

@ds5678
Copy link
Contributor

ds5678 commented Nov 29, 2025

@thegu5 Can you rebase?

@thegu5
Copy link
Contributor Author

thegu5 commented Nov 29, 2025

Fixing...

@thegu5
Copy link
Contributor Author

thegu5 commented Nov 29, 2025

Everything looks good now, I removed the original sln file since the project is built with dotnet 9 now

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.

4 participants