Surface const-sql diagnostics from generated files - #178
Merged
mgravell merged 2 commits intoAug 20, 2026
Annotations
11 warnings
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-dotnet@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Build:
test/Dapper.AOT.Test/Verifiers/Verifier.cs#L227
The member 'Verifier<TAnalyzer>.CSVerifyAsync<TCodeFix>(string, Func<Solution, ProjectId, Solution>[], DiagnosticResult[], SqlSyntax, SqlParseInputFlags, bool)' does not hide an accessible member. The new keyword is not required.
|
|
Build:
test/Dapper.AOT.Test/IncrementalCachingTests.cs#L114
Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
|
|
Build:
test/Dapper.AOT.Test/IncrementalCachingTests.cs#L94
Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
|
|
Build:
test/Dapper.AOT.Test/IncrementalCachingTests.cs#L74
Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
|
|
Build:
test/Dapper.AOT.Test/Integration/MsSqlFixture.cs#L18
'MsSqlBuilder.MsSqlBuilder()' is obsolete: 'This parameterless constructor is obsolete and will be removed. Use the constructor with the image parameter instead: https://github.com/testcontainers/testcontainers-dotnet/discussions/1470#discussioncomment-15185721.'
|
|
Build:
test/Dapper.AOT.Test/Integration/PostgresqlFixture.cs#L17
'PostgreSqlBuilder.PostgreSqlBuilder()' is obsolete: 'This parameterless constructor is obsolete and will be removed. Use the constructor with the image parameter instead: https://github.com/testcontainers/testcontainers-dotnet/discussions/1470#discussioncomment-15185721.'
|
|
Build:
test/Dapper.AOT.Test/Interceptors/SqliteUsage.input.cs#L20
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Build:
test/Dapper.AOT.Test/Interceptors/SqliteUsage.snapshot.cs#L100
Possible null reference assignment.
|
|
Build:
test/Dapper.AOT.Test/Interceptors/SqliteUsage.snapshot.cs#L97
Possible null reference assignment.
|
|
Build:
test/Dapper.AOT.Test/Verifiers/Verifier.cs#L227
The member 'Verifier<TAnalyzer>.CSVerifyAsync<TCodeFix>(string, Func<Solution, ProjectId, Solution>[], DiagnosticResult[], SqlSyntax, SqlParseInputFlags, bool)' does not hide an accessible member. The new keyword is not required.
|
background
wait
wait-all
cancel
parallel
Loading