Merge pull request #27 from AntAndriy7/working #54
Annotations
11 warnings
|
Sonar Check
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-dotnet@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L43
Dispose '_cts' when it is no longer needed.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L9
This class overrides 'GetHashCode' and should therefore also override 'Equals'.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L9
Move 'UdpClientWrapper' into a named namespace.
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L24
Dispose '_cts' when it is no longer needed.
|
|
Sonar Check:
NetSdrClientApp/Networking/IUdpClient.cs#L2
Move 'IUdpClient' into a named namespace.
|
|
Sonar Check:
NetSdrClientApp/Messages/NetSdrMessageHelper.cs#L10
Complete the task associated to this 'TODO' comment.
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L112
The variable 'ex' is declared but never used
|
|
Sonar Check:
NetSdrClientApp/Networking/UdpClientWrapper.cs#L38
The variable 'ex' is declared but never used
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L63
Cannot convert null literal to non-nullable reference type.
|
|
Sonar Check:
NetSdrClientApp/Networking/TcpClientWrapper.cs#L25
Non-nullable field '_cts' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|