Skip to content

Properly Multi-target Roslyn - #335

Merged
viceroypenguin merged 1 commit into
mainfrom
multi-target-roslyn
Sep 9, 2026
Merged

viceroypenguin merged 1 commit into
mainfrom
multi-target-roslyn

Conversation

@viceroypenguin

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 9 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8d52dd65-f095-4534-ab8f-bb1fc3ad13ad

📥 Commits

Reviewing files that changed from the base of the PR and between 9293bad and 6756dde.

📒 Files selected for processing (11)
  • .github/workflows/build.yml
  • .github/workflows/release.yml
  • Directory.Build.props
  • Directory.Packages.props
  • src/Immediate.Apis.Analyzers/Immediate.Apis.Analyzers.csproj
  • src/Immediate.Apis.CodeFixes/Immediate.Apis.CodeFixes.csproj
  • src/Immediate.Apis.Generators/Immediate.Apis.Generators.csproj
  • src/Immediate.Apis/Immediate.Apis.csproj
  • tests/Immediate.Apis.FunctionalTests.Scalar/Immediate.Apis.FunctionalTests.Scalar.csproj
  • tests/Immediate.Apis.FunctionalTests.Swagger/Immediate.Apis.FunctionalTests.Swagger.csproj
  • tests/Immediate.Apis.Tests/Immediate.Apis.Tests.csproj

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 34378612130

Coverage decreased (-12.8%) to 79.208%

Details

  • Coverage decreased (-12.8%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 194 coverage regressions across 2 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

194 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
src/Common/ITypeSymbolExtensions.cs 150 0.0%
src/Common/AttributeDataExtensions.cs 44 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 1515
Covered Lines: 1200
Line Coverage: 79.21%
Coverage Strength: 2.38 hits per line

💛 - Coveralls

@viceroypenguin
viceroypenguin merged commit 1913c62 into main Sep 9, 2026
2 of 3 checks passed
@viceroypenguin
viceroypenguin deleted the multi-target-roslyn branch September 9, 2026 16:50
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