Skip to content

Merge pull request #36 from GabrielMarquezMatte/develop #83

Merge pull request #36 from GabrielMarquezMatte/develop

Merge pull request #36 from GabrielMarquezMatte/develop #83

Triggered via push July 21, 2026 13:12
Status Success
Total duration 3m 2s
Artifacts

codeql.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Analyze (C#): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L22
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Analyze (C#): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L21
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Analyze (C#): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Provide an overridable implementation of Dispose(bool) on 'SyncRowEnumerator' or mark the type as sealed. A call to Dispose(false) should only clean up native resources. A call to Dispose(true) should clean up both managed and native resources. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1063)
Analyze (C#): src/ExcelReader.Core/Writer/Internal/CellFormatter.cs#L90
All 'WriteNumber' method overloads should be adjacent.