Skip to content
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

Add Code Generation Validation as a CI Requirement #33669

Open
2 tasks
alzimmermsft opened this issue Feb 22, 2023 · 0 comments
Open
2 tasks

Add Code Generation Validation as a CI Requirement #33669

alzimmermsft opened this issue Feb 22, 2023 · 0 comments
Labels
App Configuration Azure.ApplicationModel.Configuration Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer Cognitive - Metrics Advisor Cognitive - Text Analytics Container Registry EngSys This issue is impacting the engineering system. Epic Event Grid KeyVault Monitor Monitor, Monitor Ingestion, Monitor Query Schema Registry Search Tables

Comments

@alzimmermsft
Copy link
Member

alzimmermsft commented Feb 22, 2023

The Storage libraries had their CI updated to enforce that generated code matched what was cleanly generated (#33197), all SDKs that use code generation should update to do the same. This will validate that there are no hand-written changes to generated code that require manual intervention every time code generation is updated.

  • Copy logic used in Storage CI to places where code generation is used
  • Fix places where hand-written code was used in generated code by used the Autorest.java code customization framework
Service Library Driver Status Target Release MS
App Configuration azure-data-appconfiguration @alzimmermsft Done
Container Registry azure-containers-registry @lmolkova
Event Grid azure-messaging-eventgrid @billwert
Form Recognizer azure-ai-formrecognizer @samvaity
Key Vault azure-security-keyvault-administration @vcolin7
Key Vault azure-security-keyvault-certificates @vcolin7
Key Vault azure-security-keyvault-keys @vcolin7
Key Vault azure-security-keyvault-secrets @vcolin7
Metrics Advisor azure-ai-metricsadvisor @samvaity
Monitor azure-monitor-ingestion @jairmyree
Monitor azure-monitor-opentelemtry-exporter @lmolkova
Monitor azure-monitor-query @jairmyree
Schema Registry azure-data-schemaregistry @conniey
Search azure-search-documents @alzimmermsft Done
Storage azure-storage-blob @alzimmermsft Done
Storage azure-storage-file-datalake @alzimmermsft Done
Storage azure-storage-file-share @alzimmermsft Done
Storage azure-storage-queue @alzimmermsft Done
Tables azure-data-table @jairmyree
Text Analytics azure-ai-textanalytics @mssfang
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Configuration Azure.ApplicationModel.Configuration Client This issue points to a problem in the data-plane of the library. Cognitive - Form Recognizer Cognitive - Metrics Advisor Cognitive - Text Analytics Container Registry EngSys This issue is impacting the engineering system. Epic Event Grid KeyVault Monitor Monitor, Monitor Ingestion, Monitor Query Schema Registry Search Tables
Projects
Status: No status
Development

No branches or pull requests

1 participant