Skip to content

Conversation

@Saisang
Copy link
Collaborator

@Saisang Saisang commented Jun 27, 2025

When MAML2Yaml is not in the path of monikerMapping.json, reference content in the docset will be built by PSMD2Yaml instead of its predecessor MAML2Yaml. For details, see https://dev.azure.com/ceapex/Engineering/_workitems/edit/783982

@Saisang Saisang marked this pull request as draft June 27, 2025 07:15
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 15988c9:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • Line 0, Column 0: [Error: PSMD2Yaml_InternalError] Internal Error: PowerShell script not found: C:\Program Files\docs-pipeline\Scripts\Get-InstalledPlatyPSVersion.ps1

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@Saisang Saisang closed this Jul 2, 2025
@Saisang Saisang reopened this Jul 2, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 15988c9:

✅ Validation status: passed

File Status Preview URL Details
.openpublishing.publish.config.json ✅Succeeded
secmgmt/mapping/MAML2Yaml/monikerMapping.json ✅Succeeded n/a (file deleted or renamed)
secmgmt/mapping/monikerMapping.json ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 8b9ce13:

✅ Validation status: passed

File Status Preview URL Details
.openpublishing.publish.config.json ✅Succeeded
secmgmt/mapping/MAML2Yaml/monikerMapping.json ✅Succeeded n/a (file deleted or renamed)
secmgmt/mapping/monikerMapping.json ✅Succeeded

For more details, please refer to the build report.

@Saisang Saisang closed this Jul 2, 2025
@Saisang Saisang reopened this Jul 2, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 8b9ce13:

✅ Validation status: passed

File Status Preview URL Details
.openpublishing.publish.config.json ✅Succeeded
secmgmt/mapping/MAML2Yaml/monikerMapping.json ✅Succeeded n/a (file deleted or renamed)
secmgmt/mapping/monikerMapping.json ✅Succeeded

For more details, please refer to the build report.

@Saisang Saisang marked this pull request as ready for review July 2, 2025 18:58
@Saisang
Copy link
Collaborator Author

Saisang commented Jul 2, 2025

@mtrilbybassett
Copy link
Contributor

lgtm -- text wrapping in Inputs/Outputs to be addressed during/after week of 07/05

@mtrilbybassett mtrilbybassett merged commit f411847 into main Jul 2, 2025
2 checks passed
@Saisang Saisang deleted the revert-27-sai-addfolderformapping branch July 2, 2025 19:31
Copy link

@sdwheeler sdwheeler left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants