Skip to content

Conversation

Saisang
Copy link
Contributor

@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

Copy link

Learn Build status updates of commit 261fa7a:

❌ 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 marked this pull request as draft June 27, 2025 06:07
@Saisang Saisang closed this Jul 2, 2025
@Saisang Saisang reopened this Jul 2, 2025
Copy link

Learn Build status updates of commit 261fa7a:

⚠️ Validation status: warnings

File Status Preview URL Details
dsc/dsc-1.1/PSDesiredStateConfiguration/PSDesiredStateConfiguration.md ⚠️Warning View (dsc-1.1) Details
dsc/dsc-1.1/TOC.yml ⚠️Warning View (dsc-1.1) Details
dsc/dsc-2.0/PSDesiredStateConfiguration/PSDesiredStateConfiguration.md ⚠️Warning View (dsc-2.0) Details
dsc/dsc-2.0/TOC.yml ⚠️Warning View (dsc-2.0) Details
.openpublishing.publish.config.json ✅Succeeded
dsc/mapping/MAML2Yaml/monikerMapping.json ✅Succeeded n/a (file deleted or renamed)
dsc/mapping/monikerMapping.json ✅Succeeded

dsc/dsc-1.1/PSDesiredStateConfiguration/PSDesiredStateConfiguration.md

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'PSDesiredStateConfiguration.New-DscCheckSum'.

dsc/dsc-1.1/TOC.yml

  • Line 45, Column 12: [Warning: xref-not-found - See documentation] Cross reference not found: 'PSDesiredStateConfiguration.New-DscCheckSum'.

dsc/dsc-2.0/PSDesiredStateConfiguration/PSDesiredStateConfiguration.md

  • Line 0, Column 0: [Warning: xref-not-found - See documentation] Cross reference not found: 'PSDesiredStateConfiguration.New-DscCheckSum'.

dsc/dsc-2.0/TOC.yml

  • Line 25, Column 12: [Warning: xref-not-found - See documentation] Cross reference not found: 'PSDesiredStateConfiguration.New-DscCheckSum'.

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
Copy link

Learn Build status updates of commit 261fa7a:

✅ Validation status: passed

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

For more details, please refer to the build report.

Copy link

Learn Build status updates of commit 100cd6a:

✅ Validation status: passed

File Status Preview URL Details
.openpublishing.publish.config.json ✅Succeeded
dsc/mapping/MAML2Yaml/monikerMapping.json ✅Succeeded n/a (file deleted or renamed)
dsc/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:48
@Saisang
Copy link
Contributor Author

Saisang commented Jul 2, 2025

@sdwheeler Could you fix the warnings? The file name has capitalization issue. The PR should be good to go after this fixed.

CC @xmdanni @mtrilbybassett

@sdwheeler
Copy link
Contributor

sdwheeler commented Jul 2, 2025

@sdwheeler Could you fix the warnings? The file name has capitalization issue. The PR should be good to go after this fixed.

CC @xmdanni @mtrilbybassett

I fixed those already. You need to rebase your branch from main. See #352

@xmdanni
Copy link
Contributor

xmdanni commented Jul 2, 2025

I think @Saisang already merged the latest main content. Looking at build report, this repo ready for review and migration, preview page: https://review.learn.microsoft.com/en-us/powershell/module/psdesiredstateconfiguration/new-dscchecksum?view=dsc-1.1&branch=pr-en-us-351#common-parameters

@sdwheeler @mtrilbybassett

@Saisang
Copy link
Contributor Author

Saisang commented Jul 2, 2025

Yep it's good now. Would you approve?
@sdwheeler @mtrilbybassett @xmdanni

@sdwheeler sdwheeler merged commit c89f331 into main Jul 2, 2025
5 of 6 checks passed
@Saisang Saisang deleted the revert-336-sai-addfolderformapping branch July 2, 2025 19:08
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.

4 participants