Azure / azure-sdk-tools

Tools repository leveraged by the Azure SDK team.
MIT License
108 stars 166 forks source link

Have common contract for Docs.MS validation step #2676

Open sima-zhu opened 2 years ago

sima-zhu commented 2 years ago

We currently have return different format for different language at validation.

It brings us difficulty in checking the failure.

Will have validation function return the same thing and check the failure over there.

Failed pipeline:https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1343628&view=logs&j=f8e040b3-c0ff-5789-0eda-99daaaa3fc1b&t=26ac1571-47fc-5d52-3897-39c8bdecf4db

sima-zhu commented 2 years ago

A valuable point to follow up https://github.com/Azure/azure-sdk-tools/pull/2673#pullrequestreview-872310558

danieljurek commented 1 year ago

This issue is tracking work to "Make JS onboarding validation work the way Python and Java do"

JS was the first attempt at this validation and ran all packages in parallel each time (performance is pretty good when building in parallel). This can be addressed when upgrading to use the new converter provided by the skilling team.