Skip to content

Conversation

Gijsreyn
Copy link
Contributor

@Gijsreyn Gijsreyn commented Sep 5, 2025

PR Summary

This pull request adds a known issue when running dsc resource list --adapter Microsoft.Windows/WindowsPowerShell with the $env:PSModulePath containing a ";" at the end. It also fixes up an example, which was reported on the dsc channel (Discord).

PR Context

@Gijsreyn
Copy link
Contributor Author

Gijsreyn commented Sep 5, 2025

PSDesiredStateConfiguration v2.0.7 doesn't have this issue.

Copy link
Collaborator

@michaeltlombardi michaeltlombardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aside from a small formatting change, this documentation looks good. I'm going to file a separate issue for updating the adapter to return a specific exit code for this failure state, but that's nonblocking for this PR.

@Gijsreyn Gijsreyn force-pushed the docs-known-issue-psmodulepath branch from a8ae081 to f2b1319 Compare September 9, 2025 12:45
@Gijsreyn
Copy link
Contributor Author

Gijsreyn commented Sep 9, 2025

@michaeltlombardi - actually after PR #1095 got merged, the issue becomes obsolete. I wasn't assuming we didn't want to specify a version for example when it gets solved. I have removed the whole block. Only the example fix remains. I'll update the title.

@Gijsreyn Gijsreyn changed the title Add known issue with Windows PowerShell adapter and example fix Mixed PowerShell example fix Sep 9, 2025
Copy link
Member

@SteveL-MSFT SteveL-MSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a test config and not intended to be an example, I can still approve this as it probably makes more sense to test with WinPS but if the intent is an example, ideally we'd have a more scenario complete example we put into configurations folder

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.

3 participants