Skip to content

fix(api): stop leaking destination errors, honour the mapped prefix, guard schedule names - #261

Merged
thiagoluga merged 2 commits into
masterfrom
fix/api-correctness-batch
Aug 5, 2026
Merged

fix(api): stop leaking destination errors, honour the mapped prefix, guard schedule names#261
thiagoluga merged 2 commits into
masterfrom
fix/api-correctness-batch

test(api): address CodeQL — drop the redundant cast and document Test…

a336bb7
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 5, 2026 in 41s

Annotations

Check warning on line 131 in src/NeoReports.Core/Pipeline/ReportRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use explicit type instead of 'var'

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mSIcpe6GFspqvCa&open=AZ_Q-mSIcpe6GFspqvCa&pullRequest=261

Check warning on line 138 in src/NeoReports.Core/Pipeline/ReportRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use explicit type instead of 'var'

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mSIcpe6GFspqvCb&open=AZ_Q-mSIcpe6GFspqvCb&pullRequest=261

Check warning on line 1125 in src/Integrations/NeoReports.AspNetCore/NeoReportsEndpointRouteBuilderExtensions.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this null-forgiving operator; the compiler already knows this expression is not null here.

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mZScpe6GFspqvCc&open=AZ_Q-mZScpe6GFspqvCc&pullRequest=261

Check warning on line 164 in src/NeoReports.Core/Pipeline/ReportRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Local function has 8 parameters, which is greater than the 7 authorized.

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mSIcpe6GFspqvCY&open=AZ_Q-mSIcpe6GFspqvCY&pullRequest=261

Check warning on line 117 in src/NeoReports.Core/Pipeline/ReportRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use explicit type instead of 'var'

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mSIcpe6GFspqvCZ&open=AZ_Q-mSIcpe6GFspqvCZ&pullRequest=261

Check warning on line 1147 in src/Integrations/NeoReports.AspNetCore/NeoReportsEndpointRouteBuilderExtensions.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this null-forgiving operator; the compiler already knows this expression is not null here.

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mZScpe6GFspqvCd&open=AZ_Q-mZScpe6GFspqvCd&pullRequest=261

Check warning on line 511 in src/NeoReports.Core/Pipeline/ReportRunner.cs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code.

See more on https://sonarcloud.io/project/issues?id=lugarini_NeoReports&issues=AZ_Q-mSIcpe6GFspqvCX&open=AZ_Q-mSIcpe6GFspqvCX&pullRequest=261