Skip to content

🩹 [Patch]: Install built modules at their real version (bump Install-PSModuleHelpers)#52

Open
Marius Storhaug (MariusStorhaug) wants to merge 3 commits into
mainfrom
fix/bump-install-psmodulehelpers
Open

🩹 [Patch]: Install built modules at their real version (bump Install-PSModuleHelpers)#52
Marius Storhaug (MariusStorhaug) wants to merge 3 commits into
mainfrom
fix/bump-install-psmodulehelpers

Conversation

@MariusStorhaug

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Jul 8, 2026

Copy link
Copy Markdown
Member

What

Document-PSModule installs the built module (via Install-PSModule from Install-PSModuleHelpers) before generating docs. That helper used to copy the module into a hard-coded 999.0.0 folder, which breaks now that Process-PSModule#326 stamps the real version at build time (folder no longer matches the manifest, so Import-Module fails and Build-Docs breaks).

Change

Bump the Install-PSModuleHelpers pin to the version that installs into a folder matching the manifest ModuleVersion (PSModule/Install-PSModuleHelpers#19).

Pinned to the released tag: PSModule/Install-PSModuleHelpers#19 is merged and released, so this now pins v1.0.8.

Part of the Process-PSModule#326 refactor.

…ersion

Document-PSModule installs the built module via Install-PSModule (from Install-PSModuleHelpers). Bump the pin so the module installs into a folder matching its manifest ModuleVersion instead of the 999.0.0 placeholder, required now that the build stamps the real version (PSModule/Process-PSModule#326). Temporarily pinned to the fix branch (PSModule/Install-PSModuleHelpers#19); repin to the released tag before merge.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the composite action to use a newer Install-PSModuleHelpers commit that installs built PowerShell modules into a versioned folder matching the module manifest, preventing Import-Module failures during docs generation.

Changes:

  • Bumped the PSModule/Install-PSModuleHelpers pin from the previously used commit (v1.0.7) to a newer commit containing the “install real module version” fix.
  • Added an inline note indicating the pin is temporary and tied to Install-PSModuleHelpers#19.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

PR PSModule/Install-PSModuleHelpers#19 is merged and released as v1.0.8. Replace the TEMP branch pin with the released v1.0.8 commit SHA.
Copilot AI review requested due to automatic review settings July 9, 2026 00:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants