Skip to content

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

@learn-build-service-prod

This comment was marked as outdated.

@Saisang Saisang marked this pull request as draft July 2, 2025 08:47
@learn-build-service-prod

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

@learn-build-service-prod

This comment was marked as outdated.

@learn-build-service-prod

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

@learn-build-service-prod
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.

4 participants