You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
endpoint on SdkClientType. Should it be string | undefined? Can we have a client with no endpoint. If @server is there with a full-value, then endpoint is a required initialization property. If it is there with a default value, then it's an optional initialization property (NEW DESIGN) [tcgc] create endpoint type #474
clientaccessor or clientgetter / another name (keep it clientaccessor
What should we do for NoAuth. See what handwritten sdks do in storage, bc there is a use case for this. Just don't return credential as part of initialization
Nullability. Should it go on props / params or on the original type. Main use case for putting it on the type is for array types, to distinguish whether the array is nullable, or if its items are nullable [tcgc] move nullable onto params and responses #447
arm
fromSdkClientType
[tcgc] deprecate.arm
onSdkClientType
#504endpoint
onSdkClientType
. Should it bestring | undefined
? Can we have a client with no endpoint. If @server is there with a full-value, then endpoint is a required initialization property. If it is there with a default value, then it's an optional initialization property (NEW DESIGN) [tcgc] create endpoint type #474responses
andexceptions
(map of range to response)isError
be something inusage
? No. see if you can remove it completely from the returned tcgc type [tcgc] deprecate .isError on SdkModelType #503isFormDataType
also be something inusage
? Yes, call itMultipartFormData
[tcgc] add UsageFlags.MultipartFormData and deprecate.isFormDataType
#502Patch
inUsageFlags
[tcgc] add UsageFlags.JsonMergePatch #455. Call itJsonMergePatch
discriminatedModel
type with all of the polymorphic specific properties in it? No, let's keep it simple.clients
only show top-level clients. ANSWER: no parent client, but all clients should have initialization. If it's an operation group, access will be internal. Don't generate recursively (recommendation) [tcgc] make initialization required on all clients, add access to disambiguate #513clientaccessor
orclientgetter
/ another name (keep itclientaccessor
NoAuth
. See what handwritten sdks do in storage, bc there is a use case for this. Just don't returncredential
as part of initialization.nameInClient
to.name
https://github.com/Azure/typespec-azure/pull/445/files.generatedName
a bool and.name
a required prop [tcgc] Fix generated name #444initialOperation
->operation
forSdkLroServiceMethod
bodyParam
sourceModel
on aproperty
SdkBodyModelPropertyType`The text was updated successfully, but these errors were encountered: