Skip to content

Update Roslyn version to 5.0.0-1.25204.1 #8138

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

Merged
merged 5 commits into from
Apr 4, 2025

Conversation

dibarbet
Copy link
Member

@dibarbet dibarbet commented Apr 4, 2025

No description provided.

@dibarbet dibarbet requested a review from a team as a code owner April 4, 2025 22:17
@@ -4,7 +4,11 @@
- Debug from .csproj and .sln [#5876](https://github.com/dotnet/vscode-csharp/issues/5876)

# 2.73.x
* Bump Roslyn to 5.0.0-1.25204.1 (PR: [#8138](https://github.com/dotnet/vscode-csharp/pull/8138))
* [View Complete Diff of Changes](https://github.com/dotnet/roslyn/compare/1a06295e3cd3e57be2c79953f16db724db119a4f...ad1c386da419e0d6fe9f25a1f41e5cc2368964e7?w=1)
Copy link
Member Author

Choose a reason for hiding this comment

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

the roslyn-analyzers merge is flooding the git history and making the changelog tool barf. For now just linking the full diff.

@@ -108,6 +109,7 @@ describe(`Formatting Tests`, () => {
'using System;',
'using Options;',
'namespace Formatting;',
'',
Copy link
Contributor

Choose a reason for hiding this comment

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

🥳 the formatter not adding a new line has been a pain point for a while. Glad it's fixed!

Copy link

Choose a reason for hiding this comment

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

@ryzngard @dibarbet hey, I just saw this PR by accident. I fixed that issue with the namespace formatting in roslyn 2 weeks ago. I will look at more formatting stuff soon, so I'm curious if there are any other similar pain points that you/your team have?

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @Inurias ! I can't think of anything immediate. This usually hit me in VS when creating a new file. It would add the namespace, file header, etc but the formatting would be wrong for file scoped namespaces

@dibarbet dibarbet merged commit e3d1d11 into dotnet:main Apr 4, 2025
24 checks passed
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.

3 participants