Fixes #12986 - Add note about limitation of ps1 files in CLM#12988
Merged
Conversation
Contributor
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the module manifest documentation to call out that .ps1-based entries in module manifests don’t work under Constrained Language mode (CLM), aligning the docs with the behavior reported in #12986.
Changes:
- Add CLM limitation notes for
.ps1usage in module manifests (RootModule, ScriptsToProcess, NestedModules) across versionedabout_Module_Manifeststopics (5.1, 7.4–7.7). - Update the “How to write a PowerShell module manifest” article to point readers to
about_Module_Manifestsfor element-by-element details (and remove the in-article “Module manifest elements” table). - Minor formatting/wording adjustments and
ms.dateupdates.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| reference/docs-conceptual/developer/module/how-to-write-a-powershell-module-manifest.md | Removes the in-page manifest-elements table and redirects readers to about_Module_Manifests for detailed key documentation. |
| reference/7.7/Microsoft.PowerShell.Core/About/about_Module_Manifests.md | Adds CLM notes for .ps1 usage in RootModule/ScriptsToProcess/NestedModules and tweaks related wording. |
| reference/7.6/Microsoft.PowerShell.Core/About/about_Module_Manifests.md | Same CLM notes and wording adjustments as 7.7 for the 7.6 versioned topic. |
| reference/7.5/Microsoft.PowerShell.Core/About/about_Module_Manifests.md | Same CLM notes and wording adjustments as 7.7 for the 7.5 versioned topic. |
| reference/7.4/Microsoft.PowerShell.Core/About/about_Module_Manifests.md | Same CLM notes and wording adjustments as 7.7 for the 7.4 versioned topic. |
| reference/5.1/Microsoft.PowerShell.Core/About/about_Module_Manifests.md | Adds the same CLM limitations for Windows PowerShell 5.1’s versioned topic. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+54
to
+56
| For more information, see the parameter descriptions in the [New-ModuleManifest][10] cmdlet. For | ||
| descriptions of the keys and values you can include in a module manifest, see | ||
| [about_Module_Manifests][04]. |
Comment on lines
+949
to
+951
| > Modules specified as `.ps1` files aren't supported when the module is running | ||
| > in Constrained Language mode. The files listed can't run when the module is | ||
| > imported in that mode. |
Comment on lines
+949
to
+951
| > Modules specified as `.ps1` files aren't supported when the module is running | ||
| > in Constrained Language mode. The files listed can't run when the module is | ||
| > imported in that mode. |
Comment on lines
+949
to
+951
| > Modules specified as `.ps1` files aren't supported when the module is running | ||
| > in Constrained Language mode. The files listed can't run when the module is | ||
| > imported in that mode. |
Comment on lines
+949
to
+951
| > Modules specified as `.ps1` files aren't supported when the module is running | ||
| > in Constrained Language mode. The files listed can't run when the module is | ||
| > imported in that mode. |
Comment on lines
+948
to
+950
| > Modules specified as `.ps1` files aren't supported when the module is running | ||
| > in Constrained Language mode. The files listed can't run when the module is | ||
| > imported in that mode. |
Contributor
|
Learn Build status updates of commit 05bb1a1: ✅ Validation status: passed
For more details, please refer to the build report. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
Add note about limitation of ps1 files in CLM
PR Checklist