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

[Update autorest to version 20221109.2]: Unify discriminator implementations across generator types #32335

Merged
merged 43 commits into from
Nov 10, 2022

Conversation

archerzz
Copy link
Member

@archerzz archerzz commented Nov 7, 2022

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

Found the following two issues while doing this PR.

@m-nash
Copy link
Member

m-nash commented Nov 9, 2022

@archerzz lets get an answer to this before we push this in.

@m-nash m-nash changed the title [do not merge]: unify discriminator [Update autorest to version 20221109.2]: Unify discriminator implementations across generator types Nov 9, 2022
Copy link
Member

@JoshLove-msft JoshLove-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Event Grid changes look good.

@m-nash m-nash merged commit 4c2887c into Azure:main Nov 10, 2022
sofiar-msft pushed a commit to sofiar-msft/azure-sdk-for-net that referenced this pull request Dec 7, 2022
…tations across generator types (Azure#32335)

* unify discriminator

* update blueprint

* update cognitivelanguage

* update cosmosdb

* update databoxedge

* update databox

* update datafactory

* override not needed anymore for blueprint

* override no longer needed for cosmosdb

* override no longer needed for databoxedge

* update datashare

* update deploymentmanager

* update eventgrid

* update hybridaks

* update kusto

* update machinelearningservices

* update maps

* update monitor

* update recoverservices-backup

* update resources

* update securityinsights

* update servicefabric

* update servicefabricmanagedcluster

* update synapse

* update videoanalyzer

* update containerregistry

* update search

* update datafactory using latest code gen

* update synapse using latest code gen

* [problematic] update cognitivelanguage with latest code gen

* [problematic]: update securityinsights with latest code gen

* update cognitivelanguage

* update securitycenter

* update by the latest generator

* lock securityinsights until mgmt generation bug is fixed

* update autorest version and api definitions

* add words for hybrid aks

* update by latest generator

* update for making unknown derivations internal

* update autorest version

* update api

* update analyzers

Co-authored-by: m-nash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants