Skip to content

Update dependency Microsoft.CodeAnalysis.CSharp to v5#2428

Open
renovate[bot] wants to merge 2 commits into
mainfrom
renovate/major-roslyn
Open

Update dependency Microsoft.CodeAnalysis.CSharp to v5#2428
renovate[bot] wants to merge 2 commits into
mainfrom
renovate/major-roslyn

Conversation

@renovate

@renovate renovate Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
Microsoft.CodeAnalysis.CSharp 4.14.05.3.0 age confidence

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate Bot added the 🚚 private-dependencies Private dependency label Jun 29, 2026
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@codacy-production codacy-production Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

While the dependency update itself is technically straightforward, it introduces a significant logical inconsistency within the repository structure. The project file Rocket.Surgery.Conventions.Analyzers.roslyn4.14.csproj is explicitly named to indicate support for Roslyn 4.14. Upgrading it to 5.3.0 contradicts this intent.

This change should not be merged in its current state as it will likely break downstream consumers expecting 4.14 compatibility or create confusion regarding the project's purpose. Codacy analysis indicates the project otherwise remains up to standards.

About this PR

  • The project file being updated is named 'roslyn4.14.csproj', which strongly implies it is intended to provide compatibility specifically for Microsoft.CodeAnalysis version 4.14. Upgrading it to 5.3.0 creates a significant mismatch between the file's stated purpose and its implementation.

Test suggestions

  • Verify successful compilation of Rocket.Surgery.Conventions.Analyzers.roslyn4.14.csproj with the new dependency version.
  • Verify that analyzer behavior remains compatible with the updated Roslyn compiler version.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify successful compilation of Rocket.Surgery.Conventions.Analyzers.roslyn4.14.csproj with the new dependency version.
2. Verify that analyzer behavior remains compatible with the updated Roslyn compiler version.

TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback

<Project Sdk="Microsoft.NET.Sdk">
<ItemGroup>
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" VersionOverride="4.14.0" PrivateAssets="all" />
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" VersionOverride="5.3.0" PrivateAssets="all" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 MEDIUM RISK

Updating the 'Microsoft.CodeAnalysis.CSharp' version to 5.3.0 contradicts the project's filename and intended compatibility with Roslyn 4.14. If this project is meant to support Roslyn 4.14 environments, this change should be reverted. If Roslyn 5 support is intended, a new project should be created or this one should be renamed to reflect the new target.

Suggested change
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" VersionOverride="5.3.0" PrivateAssets="all" />
<PackageReference Include="Microsoft.CodeAnalysis.CSharp" VersionOverride="4.14.0" PrivateAssets="all" />

@github-actions github-actions Bot added this to the v15.0.1 milestone Jun 29, 2026
@github-actions github-actions Bot added the :shipit: merge Shipit! label Jun 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 92e6629. ± Comparison against base commit c15f555.

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

Labels

:shipit: merge Shipit! 🚚 private-dependencies Private dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants