Skip to content

No-op: README and cmdlet output already consistent for OperatingSystem field#6

Closed
Copilot wants to merge 1 commit intoRemediationfrom
copilot/sub-pr-5
Closed

No-op: README and cmdlet output already consistent for OperatingSystem field#6
Copilot wants to merge 1 commit intoRemediationfrom
copilot/sub-pr-5

Conversation

Copy link

Copilot AI commented Mar 13, 2026

An automated reviewer flagged that Get-IntuneDeviceLogin's README example included OperatingSystem but the cmdlet did not output it. Investigation confirmed the flag was incorrect — the cmdlet already emits OperatingSystem in its PSCustomObject, and the README example already reflects this. No code changes were required.

  • Verified: PSCustomObject in all parameter set branches (ById, ByName, ByUserPrincipalName/ByUserId) already includes OperatingSystem = $device.operatingSystem
  • Verified: README example output already lists OperatingSystem in the correct position

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI mentioned this pull request Mar 13, 2026
Copilot AI changed the title [WIP] [WIP] Address feedback on Get-IntuneDeviceLogon output formatting No-op: README and cmdlet output already consistent for OperatingSystem field Mar 13, 2026
Copilot AI requested a review from hjorslev March 13, 2026 20:45
@hjorslev hjorslev closed this Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants