Merge pull request #12 from atc-net/feature/dotnet-upgrade · atc-net/atc-opc-ua@efa9aed

Triggered via push March 25, 2024 14:10

perkops

main

Status Success

Total duration 1m 56s

Artifacts –

post-integration.yml

on: push

12 warnings

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, nelonoel/[email protected], actions/setup-dotnet@v1, actions/setup-java@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, nelonoel/[email protected], actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

merge-to-stable: src/Atc.Opc.Ua/Services/OpcUaValidationHelper.cs#L6

Refactor this method to reduce its Cognitive Complexity from 49 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

merge-to-stable: src/Atc.Opc.Ua/Services/OpcUaClient.cs#L181

Change this condition so that it does not always evaluate to 'True'. (https://rules.sonarsource.com/csharp/RSPEC-2589)

merge-to-stable: src/Atc.Opc.Ua/Services/OpcUaClientCommandExecution.cs#L86

Refactor this method to reduce its Cognitive Complexity from 27 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

merge-to-stable: src/Atc.Opc.Ua/Services/OpcUaClientReader.cs#L55

Collection-specific "Exists" method should be used instead of the "Any" extension. (https://rules.sonarsource.com/csharp/RSPEC-6605)

merge-to-stable: src/Atc.Opc.Ua.CLI/Program.cs#L37

Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)