Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add separate module handling for PowerShell Core #5458

Merged
merged 2 commits into from
Nov 26, 2024

Conversation

FabienTschanz
Copy link
Contributor

Pull Request (PR) description

This PR removes the dependency that all PowerShell modules need to be installed under C:\Program Files\WindowsPowerShell\Modules* and massively improves the installation speed if the Microsoft.PowerShell.PSResourceGet module is installed. This module is installed by default if run from PowerShell 7.4 or greater.

The improvement is inspired from issue #5433 and from my own experience with PowerShell 7. Microsoft365DSC seems to run fine with PowerShell 7, and now it doesn't require a hard dependency on any module except PnP.PowerShell to be installed under Windows PowerShell 5.1.

This Pull Request (PR) fixes the following issues

None.

Task list

  • Added an entry to the change log under the Unreleased section of the file CHANGELOG.md.
    Entry should say what was changed and how that affects users (if applicable), and
    reference the issue being resolved (if applicable).
  • Resource parameter descriptions added/updated in the schema.mof.
  • Resource documentation added/updated in README.md.
  • Resource settings.json file contains all required permissions.
  • Examples appropriately added/updated.
  • Unit tests added/updated.
  • New/changed code adheres to DSC Community Style Guidelines.

@ykuijs ykuijs merged commit ad0cbe0 into microsoft:Dev Nov 26, 2024
2 checks passed
@FabienTschanz FabienTschanz deleted the feat/powershell-core-modules branch November 26, 2024 16:08
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