Add Sample Projects Overview to README and fix CI build pipeline#684
Add Sample Projects Overview to README and fix CI build pipeline#684
Conversation
…s for each project Co-authored-by: romanett <7413710+romanett@users.noreply.github.com>
|
@copilot please fix the build Pipelines so CI passes |
Co-authored-by: romanett <7413710+romanett@users.noreply.github.com>
Fixed in commit The "Build Solutions" job was failing because The fix replaces both Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Users had no quick way to identify which sample project matched their use case (e.g., already have an external OPC UA server running and just need a client). Additionally, the "Build Solutions" CI job was failing due to a
NuGetCommand@2restore limitation with OS-conditional MSBuild properties.Proposed changes
README.md: Added a new
## Sample Projects Overviewsection with two scannable tables:Samples/) — ReferenceServer, ReferenceClient, UA Sample Server/Client (.NET 4.6), GDS Server/Client variants; each row has project link, Server/Client type badge, and a one-line description of purposeWorkshop/) — all 25 workshop projects (Aggregation, AlarmCondition, Boiler, DataAccess, DataTypes, HistoricalAccess, HistoricalEvents, Methods, PerfTest, SimpleEvents, UserAuthentication, Views, Empty) with the same format.azurepipelines/sln.yml: Fixed the "Build Solutions" CI job failure by replacingNuGetCommand@2 command: restore(classic nuget.exe CLI) withMSBuild@1 /t:Restore /p:RestoreForceEvaluate=true. The classic nuget.exe CLI does not reliably evaluate OS-conditional MSBuild properties (specifically$([MSBuild]::IsOsPlatform('Windows'))intargets.props), causingOpc.Ua.Sample.csprojto be resolved without itsnet48target framework, making it incompatible with the net48-targetingClient.Net4andServer.Net4projects (NU1201). Using MSBuild's native restore target ensures all property imports are correctly evaluated. The now-unusedNuGetToolInstaller@1step was also removed.Related Issues
Types of changes
Checklist
Further comments
Table entries were verified against actual directories in the repo; references to
NetCoreConsoleServer/NetCoreConsoleClient(present in existing README prose but no longer in the repo) were intentionally excluded from the new tables.The CI fix uses
MSBuild@1 /t:Restoreinstead ofNuGetCommand@2 command: restorebecause MSBuild's restore target fully evaluates all MSBuild property imports — including the OS-conditionalDotNetTargetFrameworkproperty that addsnet48;toLibTargetFrameworkson Windows — whereas classicnuget.exe restoremay not evaluate these conditions consistently across tool versions.Original prompt
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.