Skip to content

Update Start-ManagedFolderAssistant.md #11546

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

Merged
merged 3 commits into from
Dec 22, 2023
Merged

Conversation

Logonzal
Copy link
Contributor

Add opening and closing square bracket in the syntax example.

Add opening and closing square bracket in the syntax example.
Copy link

Learn Build status updates of commit 6f34c54:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Start-ManagedFolderAssistant.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 2a645a9:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Start-ManagedFolderAssistant.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

@chrisda
Copy link
Contributor

chrisda commented Dec 22, 2023

@Logonzal, you're right, but strange things are going on in that cmdlet.

The platyPS utility we use to dump cmdlets to markdown is detecting the HoldCleanup and StopHoldCleanUp switch parameters as Required: True in the HoldCleanupParameterSet and StopHoldCleanupParameterSet parameter sets. When I checked Exchange on-prem and Online code, those parameters are definitely tagged as Required: False.

If a switch parameter is required, by definition it can't have brackets around it, because brackets = optional, which is likely why at some point I removed the brackets. Why I also didn't remove them from StopHoldCleanup is unclear.

Unclear what's going on (I've never known platyPS to make mistakes like that ?!?), but I used my audit from last month to modify your PR to make sure everything matches current reality, including adding square brackets to HoldCleanup and the other ramifications of Required: False.

Copy link

Learn Build status updates of commit 0760890:

✅ Validation status: passed

File Status Preview URL Details
exchange/exchange-ps/exchange/Start-ManagedFolderAssistant.md ✅Succeeded View (exchange-ps)

For more details, please refer to the build report.

For any questions, please:

@chrisda chrisda merged commit a799c74 into MicrosoftDocs:main Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants