Skip to content

Conversation

alekyaj
Copy link
Contributor

@alekyaj alekyaj commented Feb 19, 2025

The update here is actually an extension of task 9800612 and the SME wants the note to be in the standard format. I've updated the same.

Copy link

Learn Build status updates of commit a74ba0a:

✅ Validation status: passed

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

For more details, please refer to the build report.

For any questions, please:

@alekyaj alekyaj added Sign off The pull request is ready to be reviewed and merged by PubOps and removed Do not merge labels Feb 19, 2025
@padmagit77 padmagit77 merged commit b5c30e5 into main Feb 19, 2025
11 checks passed
@padmagit77 padmagit77 self-assigned this Feb 19, 2025
@padmagit77 padmagit77 added the In review PubOps is reviewing the pull request label Feb 19, 2025
@chrisda
Copy link
Contributor

chrisda commented Feb 19, 2025

@alekyaj, any special formatting like Alerts is lost when the markdown files are combined and converted to XML for code check-in for help at the command line (Get-Help ). They look great on the web, not so great in ASCII text. There was also a period of time where some areas of the cmdlet reference article didn't support Alert formatting (the SYNOPSIS), so I simply don't use it.

@alekyaj
Copy link
Contributor Author

alekyaj commented Feb 20, 2025

@alekyaj, any special formatting like Alerts is lost when the markdown files are combined and converted to XML for code check-in for help at the command line (Get-Help ). They look great on the web, not so great in ASCII text. There was also a period of time where some areas of the cmdlet reference article didn't support Alert formatting (the SYNOPSIS), so I simply don't use it.

@chrisda, Noted! The request to keep the note in standard markdown came from the PM, so I made the change. I will definitely keep this in mind moving forward. Thanks for the detailed explanation. Please let me know if you want me to revert the note. Thanks

@chrisda
Copy link
Contributor

chrisda commented Feb 20, 2025

@alekyaj, no need to revert. I might change it in the future.

@github-actions github-actions bot deleted the aljupudi-9800612-noteformat branch June 1, 2025 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In review PubOps is reviewing the pull request Microsoft submitter Sign off The pull request is ready to be reviewed and merged by PubOps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants