Skip to content

Migrate docs build to use PSMD2Yaml and new page template #3667

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 6 commits into from
Jul 11, 2025

Conversation

Saisang
Copy link
Contributor

@Saisang Saisang commented Jul 2, 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

This comment was marked as outdated.

@Saisang Saisang marked this pull request as draft July 2, 2025 08:47

This comment was marked as outdated.

@Saisang
Copy link
Contributor Author

Saisang commented Jul 2, 2025

@sdwheeler Could you fix the errors?

  1. "yaml-syntax-error" type error? That's because the metadata has extra "{{}}".
  2. azps-12.5.0/Az.CognitiveServices/New-AzCognitiveServicesAccountDeployment.md, Missing required attribute: 'parameters.cmdletParameters.parameterSets.position'.

And the warnings:

  1. xref-not-found, File name issue.
  2. PSMD2Yaml_InconsistentCmdletsInModule, "Inconsistent cmdlets found in module: Az.Network. 2 cmdlets in the module folder but not listed in the module file: New-AzApplicationGatewayFirewallPolicyException, New-AzPacketCaptureSettingsConfig."

The other errors/warnings would be fixed in next batch by xmdanni.

CC @xmdanni @mtrilbybassett

@sdwheeler
Copy link
Collaborator

See fixes in #3670

This comment was marked as outdated.

This comment was marked as outdated.

@Saisang
Copy link
Contributor Author

Saisang commented Jul 8, 2025

@sdwheeler Hi Sean, there is one warning left, would you please refer to this build log and fix it? That's caused by file name case.

CC @mikefrobbins

@sdwheeler
Copy link
Collaborator

@sdwheeler Hi Sean, there is one warning left, would you please refer to this build log and fix it? That's caused by file name case.

CC @mikefrobbins

This should be fixed by #3679

Copy link
Contributor

Learn Build status updates of commit f580079:

✅ Validation status: passed

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

For more details, please refer to the build report.

@Saisang
Copy link
Contributor Author

Saisang commented Jul 11, 2025

@Saisang Saisang marked this pull request as ready for review July 11, 2025 07:40
@Saisang Saisang merged commit 4a04ef6 into main Jul 11, 2025
4 checks passed
@Saisang Saisang deleted the Saisang-patch-1 branch July 11, 2025 08:36
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.

3 participants