Update ExcelReader.cs ReadRows 的时候变量应该循环 (#137) #266
Annotations
1 error and 21 warnings
test
Process completed with exit code 1.
|
test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
test:
Test/Program.cs#L35
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
test:
Test/Program.cs#L408
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
test:
Test/Program.cs#L96
The field 'Program._client' is never used
|
test:
XUnitTest.Core/Collections/CollectionHelperTests.cs#L37
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
test:
XUnitTest.Core/Collections/CollectionHelperTests.cs#L47
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
test:
XUnitTest.Core/Log/TracerTests.cs#L401
'WebRequest.CreateHttp(string)' is obsolete: 'WebRequest, HttpWebRequest, ServicePoint, and WebClient are obsolete. Use HttpClient instead.' (https://aka.ms/dotnet-warnings/SYSLIB0014)
|
test:
XUnitTest.Core/IO/ExcelReaderTests.cs#L40
Public method 'Test2' on test class 'ExcelReaderTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
test:
XUnitTest.Core/Extension/StringHelperTests.cs#L86
Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013)
|
test:
XUnitTest.Core/Data/PacketTests.cs#L178
The literal or constant value (byte)0x11 should be passed as the 'expected' argument in the call to 'Assert.Equal(expected, actual)' in method 'AfterAppendSetTest' on type 'PacketTests'. Swap the parameter values. (https://xunit.net/xunit.analyzers/rules/xUnit2000)
|
test:
XUnitTest.Core/Data/PacketTests.cs#L224
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
|
test:
Test/Program.cs#L35
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
test:
Test/Program.cs#L408
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
test:
Test/Program.cs#L96
The field 'Program._client' is never used
|
test:
XUnitTest.Core/Collections/CollectionHelperTests.cs#L37
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
test:
XUnitTest.Core/Collections/CollectionHelperTests.cs#L47
The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
|
test:
XUnitTest.Core/Log/TracerTests.cs#L401
'WebRequest.CreateHttp(string)' is obsolete: 'WebRequest, HttpWebRequest, ServicePoint, and WebClient are obsolete. Use HttpClient instead.' (https://aka.ms/dotnet-warnings/SYSLIB0014)
|
test:
XUnitTest.Core/IO/ExcelReaderTests.cs#L40
Public method 'Test2' on test class 'ExcelReaderTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
test:
XUnitTest.Core/Common/MachineInfoTests.cs#L41
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
|
test:
XUnitTest.Core/Extension/StringHelperTests.cs#L86
Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013)
|
test:
XUnitTest.Core/Data/FlowIdTests.cs#L204
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
|