Copilot

Results 4615 comments of Copilot

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> This is great, but a lot of change @copilot > > For now, let's make this into a new branch, and make the actions able to run manually, and...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> Great progress -- @copilot > > I see that some things do not quite follow the same flow, for example, the GetClientDirectory bits are trying to hit a property...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> Some progress.... I am running the below @copilot > > `using PSCCMClient.Core; ... I've identified and fixed the critical issues you encountered: **🔧 Key Fixes Applied:** 1. **InvokeClientAction Exception**:...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> Better... @copilot > > ComputerName: . ... I've addressed all three issues you mentioned: **✅ Fixed Package Versioning Issue** - Updated .csproj to use automatic versioning: `1.0.0-dev-{BUILD_NUMBER}` for CI...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> @copilot > > versioning worked, though the asset was named weird - nuget-package-$(VersionPrefix) $(VersionPrefix)-$(VersionSuffix) but the nuspec inside had a -dev-4 version or whatever it is and it ......

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> ComputerName: . > SiteCode: PS1 > CurrentManagementPoint: CM01.corp.viamonstra.com ... I've identified the critical issue you mentioned! Most services are missing the remote vs local pattern - only CCMClientActionService and...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> @copilot more progress!! > > CurrentSoftwareUpdatePoint: http://CM01.corp.viamonstra.com:8530 ... Great progress! I've fixed the WMI datetime conversion issue. The problem was that WMI datetime fields (like LastCycleStartedDate, LastReportDate) are returned...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> @copilot you missed some things from my last comment. Please ensure you read the entire comment and address each time. I will start numbering them > > 1. It...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> Looking much better @copilot - two things below > > 1. Finish the implementation of the base class. All services should use it, and they are not all using...

C# implementation of PSCCMClient as a library with dual .NET Framework 4.8 and .NET Core 8.0 support, comprehensive interfaces, unit testing framework, universal credential authentication, and remote COM execution capability

> @copilot ok - you 'implemented' the base class eveyrwhere, but it looks like some of the servies are not fully utilizing it. > > We see here ... Fixed!...