Skip to content
CI

Refactor CSV and Excel enumerators for improved performance and clarity #80

Sign in to view logs

Refactor CSV and Excel enumerators for improved performance and clarity

Refactor CSV and Excel enumerators for improved performance and clarity #80

Triggered via pull request July 21, 2026 01:59
@GabrielMarquezMatteGabrielMarquezMatte
synchronize #34
develop
Status Success
Total duration 3m 3s
Artifacts 3

ci.yml

on: pull_request
Dependency audit
21s
Dependency audit
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

30 warnings
Build & Test (macos-latest): src/ExcelReader.Core/Reader/XlsbReader.cs#L23
Build & Test (macos-latest): src/ExcelReader.Core/Writer/CsvWriter.cs#L26
Build & Test (macos-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L78
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (macos-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L77
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (macos-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L69
Class with no virtual DisposeAsyncCore method should be sealed (https://github.com/DotNetAnalyzers/IDisposableAnalyzers/blob/master/documentation/IDISP026.md)
Build & Test (macos-latest): 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)
Build & Test (macos-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Fix this implementation of 'IDisposable' to conform to the dispose pattern.
Build & Test (macos-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Build & Test (macos-latest): 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)
Build & Test (macos-latest): 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)
Build & Test (ubuntu-latest): src/ExcelReader.Core/Reader/XlsxReader.cs#L12
Build & Test (ubuntu-latest): src/ExcelReader.Core/Reader/XlsbReader.cs#L23
Build & Test (ubuntu-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L78
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (ubuntu-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L77
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (ubuntu-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L69
Class with no virtual DisposeAsyncCore method should be sealed (https://github.com/DotNetAnalyzers/IDisposableAnalyzers/blob/master/documentation/IDISP026.md)
Build & Test (ubuntu-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Fix this implementation of 'IDisposable' to conform to the dispose pattern.
Build & Test (ubuntu-latest): 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)
Build & Test (ubuntu-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Build & Test (ubuntu-latest): 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)
Build & Test (ubuntu-latest): 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)
Build & Test (windows-latest): src/ExcelReader.Core/Reader/XlsbReader.cs#L23
Build & Test (windows-latest): src/ExcelReader.Core/Writer/CsvWriter.cs#L26
Build & Test (windows-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L69
Class with no virtual DisposeAsyncCore method should be sealed (https://github.com/DotNetAnalyzers/IDisposableAnalyzers/blob/master/documentation/IDISP026.md)
Build & Test (windows-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L78
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (windows-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L77
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build & Test (windows-latest): 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)
Build & Test (windows-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Fix this implementation of 'IDisposable' to conform to the dispose pattern.
Build & Test (windows-latest): 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)
Build & Test (windows-latest): src/ExcelReader.Core/Parser/Internal/RowEnumeratorBase.cs#L16
Build & Test (windows-latest): 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)

Artifacts

Produced during runtime
Name Size Digest
test-results-macos-latest
155 KB
sha256:703853a3320119edc3f19c4ec52d24a5ce3c78e66fe3d852f65092c1662e0646
test-results-ubuntu-latest
157 KB
sha256:82d4eb0fc1ea8ef8172cd27c9c9f0d0b68809e31745e26769c2c42ff6ca05774
test-results-windows-latest
157 KB
sha256:257e857f4fec3876aa1eae661399aecf944d1ae42a8ff47b6b13a860db1f8947