Refactor Excel writers and readers for performance and limit handling #29
ci.yml
on: pull_request
Dependency audit
13s
Matrix: test
Annotations
30 warnings and 1 notice
|
Build & Test (macos-latest):
src/ExcelReader.Core/Writer/XlsbSheetWriter.cs#L65
Call System.IDisposable.Dispose on object created by 'new XlsbRowWriter(this, _date1904)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L21
Disposable field _rowWriter in type SheetWriter is not disposed (https://github.com/Vannevelj/SharpSource/blob/master/docs/SS066-DisposableFieldIsNotDisposed.md)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Parser/Internal/ExcelEnumerable.cs#L125
Method 'ExcelReader.Core.Parser.Internal.ExcelEnumerable<T, TReader, TEnumerator>.AsyncEnumerator.AsyncEnumerator(TReader, ExcelReader.Core.Parser.Internal.TypeMapInfo<T>, System.StringComparer, ExcelReader.Core.Parser.HeaderNormalization, int, System.Threading.CancellationToken, System.IFormatProvider)' should take CancellationToken as the last parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1068)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Reader/XlsbReader.Enumerator.cs#L23
'Enumerator' contains field '_reader' that is of IDisposable type 'XlsbReader', but it is never disposed. Change the Dispose method on 'Enumerator' to call Close or Dispose on this field. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2213)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
Call System.IDisposable.Dispose on object created by 'GetAsyncEnumerator(ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
'XlsReader.GetAsyncEnumerator(CancellationToken)' synchronously blocks. Await 'XlsReader.GetAsyncEnumeratorAsync(CancellationToken)' instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1849)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L116
Call System.IDisposable.Dispose on object created by 'new Enumerator(this, _sheets[_current].Offset, ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L85
Use "Async" suffix in names of methods that return an awaitable type (https://microsoft.github.io/vs-threading/analyzers/VSTHRD200.html)
|
|
|
|
Build & Test (macos-latest):
src/ExcelReader.Core/Writer/XlsbSheetWriter.cs#L65
Call System.IDisposable.Dispose on object created by 'new XlsbRowWriter(this, _date1904)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Writer/XlsbSheetWriter.cs#L65
Call System.IDisposable.Dispose on object created by 'new XlsbRowWriter(this, _date1904)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L21
Disposable field _rowWriter in type SheetWriter is not disposed (https://github.com/Vannevelj/SharpSource/blob/master/docs/SS066-DisposableFieldIsNotDisposed.md)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Reader/XlsbReader.Enumerator.cs#L23
'Enumerator' contains field '_reader' that is of IDisposable type 'XlsbReader', but it is never disposed. Change the Dispose method on 'Enumerator' to call Close or Dispose on this field. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2213)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
Call System.IDisposable.Dispose on object created by 'GetAsyncEnumerator(ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
'XlsReader.GetAsyncEnumerator(CancellationToken)' synchronously blocks. Await 'XlsReader.GetAsyncEnumeratorAsync(CancellationToken)' instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1849)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L116
Call System.IDisposable.Dispose on object created by 'new Enumerator(this, _sheets[_current].Offset, ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Writer/XlsbSheetWriter.cs#L65
Call System.IDisposable.Dispose on object created by 'new XlsbRowWriter(this, _date1904)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L85
Use "Async" suffix in names of methods that return an awaitable type (https://microsoft.github.io/vs-threading/analyzers/VSTHRD200.html)
|
|
Build & Test (ubuntu-latest):
src/ExcelReader.Core/Parser/Internal/ExcelEnumerable.cs#L125
Method 'ExcelReader.Core.Parser.Internal.ExcelEnumerable<T, TReader, TEnumerator>.AsyncEnumerator.AsyncEnumerator(TReader, ExcelReader.Core.Parser.Internal.TypeMapInfo<T>, System.StringComparer, ExcelReader.Core.Parser.HeaderNormalization, int, System.Threading.CancellationToken, System.IFormatProvider)' should take CancellationToken as the last parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1068)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Parser/Internal/ExcelEnumerable.cs#L125
Method 'ExcelReader.Core.Parser.Internal.ExcelEnumerable<T, TReader, TEnumerator>.AsyncEnumerator.AsyncEnumerator(TReader, ExcelReader.Core.Parser.Internal.TypeMapInfo<T>, System.StringComparer, ExcelReader.Core.Parser.HeaderNormalization, int, System.Threading.CancellationToken, System.IFormatProvider)' should take CancellationToken as the last parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1068)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L21
Disposable field _rowWriter in type SheetWriter is not disposed (https://github.com/Vannevelj/SharpSource/blob/master/docs/SS066-DisposableFieldIsNotDisposed.md)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Reader/XlsbReader.Enumerator.cs#L23
'Enumerator' contains field '_reader' that is of IDisposable type 'XlsbReader', but it is never disposed. Change the Dispose method on 'Enumerator' to call Close or Dispose on this field. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2213)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
Call System.IDisposable.Dispose on object created by 'GetAsyncEnumerator(ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L121
'XlsReader.GetAsyncEnumerator(CancellationToken)' synchronously blocks. Await 'XlsReader.GetAsyncEnumeratorAsync(CancellationToken)' instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1849)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Reader/XlsReader.cs#L116
Call System.IDisposable.Dispose on object created by 'new Enumerator(this, _sheets[_current].Offset, ct)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Writer/XlsbSheetWriter.cs#L65
Call System.IDisposable.Dispose on object created by 'new XlsbRowWriter(this, _date1904)' before all references to it are out of scope (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2000)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Writer/SheetWriter.cs#L85
Use "Async" suffix in names of methods that return an awaitable type (https://microsoft.github.io/vs-threading/analyzers/VSTHRD200.html)
|
|
Build & Test (windows-latest):
src/ExcelReader.Core/Parser/Internal/ExcelEnumerable.cs#L125
Method 'ExcelReader.Core.Parser.Internal.ExcelEnumerable<T, TReader, TEnumerator>.AsyncEnumerator.AsyncEnumerator(TReader, ExcelReader.Core.Parser.Internal.TypeMapInfo<T>, System.StringComparer, ExcelReader.Core.Parser.HeaderNormalization, int, System.Threading.CancellationToken, System.IFormatProvider)' should take CancellationToken as the last parameter (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1068)
|
|
Build & Test (macos-latest)
The macos-latest label will migrate to macOS 26 beginning June 15, 2026. For more information see https://github.com/actions/runner-images/issues/14167
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
test-results-macos-latest
|
84.9 KB |
sha256:229a5dcd2755d6b033e0e6df4dbe79f94f0ae5afa82e5988fd1d450e2e5aeefe
|
|
|
test-results-ubuntu-latest
|
86.1 KB |
sha256:cee5ab7cba692218551c6dab93d1b3a4aceb2a3eedef746ebd5d19535598220a
|
|
|
test-results-windows-latest
|
86 KB |
sha256:18a8e5c705f8f5ade345c9bbbce35ce3f8b45dae1f14d57bc3788328477fe79f
|
|