Skip to content

Conversation

@AndriySvyryd
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings January 8, 2026 20:18
@AndriySvyryd AndriySvyryd changed the title Disable Analyzer tests [release/8.0] Disable Analyzer tests Jan 8, 2026
@AndriySvyryd AndriySvyryd requested a review from a team January 8, 2026 20:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to disable analyzer tests in the EFCore.Analyzers.Tests project by changing test class visibility from public to internal, referencing Issue #37106 as the reason for this change.

Key Changes

  • Changed three test classes from public to internal to prevent test execution
  • Added comments referencing Issue #37106
  • No actual test logic or functionality was modified

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
UninitializedDbSetDiagnosticSuppressorTests.cs Changed class visibility to internal and added issue reference comment
InterpolatedStringUsageInRawQueriesTests.cs Changed class visibility to internal and added issue reference comment
InternalUsageDiagnosticAnalyzerTests.cs Changed class visibility to internal and added issue reference comment

@AndriySvyryd AndriySvyryd enabled auto-merge (squash) January 8, 2026 20:33
@AndriySvyryd AndriySvyryd merged commit f164971 into release/8.0 Jan 8, 2026
12 of 13 checks passed
@AndriySvyryd AndriySvyryd deleted the DisableAnalyzerTests_8.0 branch January 8, 2026 23:11
This was referenced Feb 10, 2026
This was referenced Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants