Improve documentation and functionality for PnP PowerShell - #5483
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The GCC endpoint, retry documentation, ineffective -WhatIf, and release-workflow safety issues must be corrected.
Review effort: Balanced
Findings: 6
Open (14)
Pin stable release builds to the trusted main branch · New Pin ARM32 release checkout to main · New Pin ARM64 release checkout to main · New Pin AMD64 release checkout to main · New Honor ShouldProcess for WhatIf and Confirm · New Use the worldwide endpoint for GCC connections · New Complete release assets before publishing the module · New Limit sovereign endpoint changes to GCC High and DoD · New Document the correct GCC Office Management endpoint · New Fix the spelling of equivalent · New Clarify SharePointPnPHttpTimeout scope · New Correct the GCC endpoint mapping · New Correct retry behavior documentation for CSOM requests · New Clarify timeout settings do not affect CSOM requests · New
What changed in this PR
Improves sovereign-cloud connectivity, operational documentation, and release/container automation.
Changes:
- Adds cloud-aware authentication and Office Management API endpoints.
- Expands logging, throttling, MCP, cloud, and cmdlet documentation.
- Updates Docker and release workflows, including SBOM/provenance generation.
| File | Description |
|---|---|
src/Commands/Utilities/Auth/Endpoints.cs |
Adds cloud-specific Management API endpoints. |
src/Commands/Features/DisableFeature.cs |
Deprecates -Force. |
src/Commands/Base/PnPOfficeManagementApiCmdlet.cs |
Uses connection-specific endpoints. |
src/Commands/Base/PnPConnection.cs |
Preserves cloud and Graph endpoint state. |
src/Commands/Base/ConnectOnline.cs |
Extends workload identity cloud options. |
README.md |
Links MCP server documentation. |
pages/articles/toc.yml |
Registers new articles. |
pages/articles/throttling.md |
Documents retry behavior. |
pages/articles/registerapplication.md |
Updates national-cloud registration guidance. |
pages/articles/nationalclouds.md |
Adds sovereign-cloud guidance. |
pages/articles/mcpserver.md |
Documents MCP integration. |
pages/articles/logging.md |
Adds tracing guidance. |
pages/articles/environmentvariables.md |
Documents additional settings. |
pages/articles/docker.md |
Refreshes image and architecture guidance. |
pages/articles/determinepermissions.md |
Replaces verbose guidance with tracing. |
pages/articles/azurefunctions.md |
Updates runtime and module guidance. |
pages/articles/azureautomationrunbook.md |
Updates runtime-environment setup. |
pages/articles/authentication.md |
Corrects national-cloud guidance. |
documentation/Write-PnPTraceLog.md |
Corrects cmdlet syntax and metadata. |
documentation/Stop-PnPTraceLog.md |
Clarifies parameter defaults. |
documentation/Move-PnPItemProxy.md |
Adds cmdlet reference documentation. |
documentation/Get-PnPUnifiedAuditLog.md |
Documents cloud endpoints. |
documentation/Get-PnPTraceLog.md |
Clarifies log-stream availability. |
documentation/Disable-PnPFeature.md |
Documents obsolete -Force. |
documentation/Connect-PnPOnline.md |
Updates parameter-set documentation. |
CHANGELOG.md |
Records user-facing changes. |
.github/workflows/stabledockerimages.yml |
Reworks stable image publication. |
.github/workflows/nightlydockerimages.yml |
Passes the nightly ARM32 version. |
.github/workflows/majorrelease.yml |
Adds release assets and attestations. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.



Type
Related Issues?
Fixes #NA
What is in this Pull Request ?