OTOPCUA0001: 523 unsuppressed analyzer warnings across 13 test projects #595

New Issue

2026-10-08T02:31:09-04:00

dohertj2 commented

2026-10-08 02:31:09 -04:00

Directory.Build.props says projects that intentionally call capabilities unwrapped get a scoped NoWarn; 11 test projects have it, 13 don't: Modbus.Tests 115, TwinCAT.Tests 83, FOCAS.Tests 73, AbLegacy.Tests 66, Galaxy.Tests 58, Modbus.IntegrationTests 37, Mqtt.Tests 28, OpcUaClient.Tests 21, TwinCAT.IntegrationTests 18, FOCAS.IntegrationTests 10, Calculation.Tests 6, OpcUaClient.IntegrationTests 5, AbLegacy.IntegrationTests 3. None in src/. The noise hides a real regression.
Sources: Directory.Build.props:19-28; dotnet build --no-incremental on master 2026-10-08.

Add the documented scoped <NoWarn> (with the arch-review 07/C-1 comment) to each, or wrap the calls
Consider failing the build on OTOPCUA0001 in src/

Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep.

Directory.Build.props says projects that intentionally call capabilities unwrapped get a scoped NoWarn; 11 test projects have it, 13 don't: Modbus.Tests 115, TwinCAT.Tests 83, FOCAS.Tests 73, AbLegacy.Tests 66, Galaxy.Tests 58, Modbus.IntegrationTests 37, Mqtt.Tests 28, OpcUaClient.Tests 21, TwinCAT.IntegrationTests 18, FOCAS.IntegrationTests 10, Calculation.Tests 6, OpcUaClient.IntegrationTests 5, AbLegacy.IntegrationTests 3. None in `src/`. The noise hides a real regression. Sources: Directory.Build.props:19-28; `dotnet build --no-incremental` on master 2026-10-08. - [ ] Add the documented scoped `<NoWarn>` (with the arch-review 07/C-1 comment) to each, or wrap the calls - [ ] Consider failing the build on OTOPCUA0001 in `src/` _Filed 2026-10-08 by the pending/deferred-work sweep after the #562–#591 issue sweep._

dohertj2 added the cross-driver label 2026-10-08 02:31:09 -04:00

dohertj2 referenced this issue from a commit

2026-10-08 09:03:17 -04:00

build: scoped OTOPCUA0001 NoWarn in 13 test projects, error under src/, pin SDK via global.json (#595 #606)

dohertj2 commented

2026-10-08 09:24:29 -04:00

Resolved on master (7aa77588, merge 792510c0).

The scoped <NoWarn>OTOPCUA0001</NoWarn> (with the arch-review 07/C-1 rationale comment) is now in all 13 listed test projects.
OTOPCUA0001 is an error under src/ (WarningsAsErrors in Directory.Build.props). Also pinned the SDK via global.json (#606).
Full build is clean (0 errors) and the full suite on 5a5f9b22 was green.

Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).

Resolved on master (7aa77588, merge 792510c0). - [x] The scoped `<NoWarn>OTOPCUA0001</NoWarn>` (with the arch-review 07/C-1 rationale comment) is now in all 13 listed test projects. - [x] OTOPCUA0001 is an error under `src/` (`WarningsAsErrors` in Directory.Build.props). Also pinned the SDK via global.json (#606). - Full build is clean (0 errors) and the full suite on 5a5f9b22 was green. *Closed 2026-10-08 by the hardware-available issue sweep (master d898262c).*

dohertj2 closed this issue

2026-10-08 09:24:29 -04:00

dohertj2 referenced this issue

2026-10-08 09:24:37 -04:00

Engineering system: CI evidence, global.json, regen-readme check, SlowUnit split (archreview r3 item 12) #606

dohertj2 referenced this issue

2026-10-13 16:03:28 -04:00

Offline CI retains 417 distinct test-project build warnings #725

Sign in to join this conversation.