Skip to content

Update dependency Microsoft.CodeAnalysis.CSharp to v5 - #1533

Open
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-roslyn
Open

Update dependency Microsoft.CodeAnalysis.CSharp to v5#1533
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/major-roslyn

Conversation

@renovate

@renovate renovate Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
Microsoft.CodeAnalysis.CSharp 4.14.05.6.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 24, 2026
@github-actions github-actions Bot added the :shipit: merge Shipit! label Jun 24, 2026
@codacy-production

codacy-production Bot commented Jun 24, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

Test Results

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

Results for commit 3987c89. ± Comparison against base commit 08c08b9.

♻️ This comment has been updated with latest results.

@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

The update to Roslyn 5.3.0 satisfies the requirement to bump the package version. However, there is a significant architectural misalignment as the update occurs within the 'roslyn4.6' project. This inconsistency between the project naming convention and its implementation violates the expected structure for multi-targeting Roslyn versions and may lead to consumer confusion. No security flaws or major logic bugs were identified, but the naming conflict should be resolved prior to merging to maintain structural integrity.

About this PR

  • There is a naming misalignment between the project directory/assembly name and the updated dependency version. Updating the package to 5.3.0 within a project explicitly labeled 'roslyn4.6' creates a structural contradiction that impacts the clarity of the solution's versioning strategy.

Test suggestions

  • Verify that the project compiles and functions correctly with Roslyn 5.3.0
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that the project compiles and functions correctly with Roslyn 5.3.0

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

<ItemGroup>
<PackageReference Include="Polyfill" PrivateAssets="All" />
<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

The project naming 'Rocket.Surgery.MyAssembly.roslyn4.6' is inconsistent with the update to 'Microsoft.CodeAnalysis.CSharp' version 5.3.0. This conflict between the project's identity and its implementation can confuse consumers and likely violates the architectural pattern for version-specific implementations in this solution. Consider whether a new project should be established for 5.x or if the current project should be renamed.

@renovate renovate Bot changed the title Update roslyn (major) Update dependency Microsoft.CodeAnalysis.CSharp to v5 Jun 25, 2026
@renovate
renovate Bot force-pushed the renovate/major-roslyn branch from a621a9b to 3aab719 Compare July 2, 2026 22:14
@github-actions github-actions Bot added this to the v1.3.0 milestone Jul 2, 2026
@renovate
renovate Bot force-pushed the renovate/major-roslyn branch from 3aab719 to 0f9f7cc Compare July 12, 2026 12:32
@renovate
renovate Bot force-pushed the renovate/major-roslyn branch from 0f9f7cc to 7dd4538 Compare July 21, 2026 01:56
@renovate
renovate Bot force-pushed the renovate/major-roslyn branch from 7dd4538 to 3987c89 Compare July 30, 2026 20:00
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