Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.69k stars 5.11k forks source link

Java specific renames for Document Intelligence 2024-11-30 GA #31595

Open samvaity opened 3 days ago

samvaity commented 3 days ago

I've made Java-specific updates in the pull request at https://github.com/Azure/azure-rest-api-specs/pull/31522. Here are a few notes on the typespec generation:

  1. Renaming URL and base64 source fields in ClassifyDocumentRequest and AnalyzeDocumentRequest for Java doesn't update the corresponding documentation.

  2. Changing model names creates new models without deleting the old ones.

  3. Need advice on renaming AnalyzeDocumentFromBase64 to AnalyzeDocumentFromBytes and updating its content to exclude base64 references.

@srnagar, @weidongxu-microsoft, could you provide any insights on these points?

openapi-pipeline-app[bot] commented 3 days ago

Next Steps to Merge

Next steps that must be taken to merge this PR:
openapi-pipeline-app[bot] commented 3 days ago

Generated ApiView

Language Package Name ApiView Link
JavaScript @azure-rest/ai-document-intelligence https://apiview.dev/Assemblies/Review/0ca4eda857b94249aa787305a387271f?revisionId=df668de4887f4e5980a60120463974f9
Python azure-ai-documentintelligence https://apiview.dev/Assemblies/Review/e537ef4f04a343e1863478cb96d48b59?revisionId=3b3a9bab7e794c35a7ca70581d4280a1
TypeSpec DocumentIntelligence https://apiview.dev/Assemblies/Review/f8a22c44fd034f7597acfffdba53dade?revisionId=81599968a1bb42f8946c72f6e0c7a143
azure-sdk commented 3 days ago

API change check

APIView has identified API level changes in this PR and created following API reviews.

Microsoft.App Microsoft.Authorization Microsoft.DocumentDB preview

weidongxu-microsoft commented 2 days ago

For (2), if there is no customization in your SDK, you may just delete the src/main folder to clean up the previous files, then tsp-client update to generate new (emitter does not know whether you have customization, so it won't do this for you).