Azure / autorest.go

Extension for AutoRest (https://github.com/Azure/autorest) that generates Go code
MIT License
65 stars 42 forks source link

operations client list operation #1330

Closed Alancere closed 4 weeks ago

Alancere commented 1 month ago

tsp: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/standbypool/StandbyPool.Management/service.tsp#L35

jhendrixMSFT commented 1 month ago

This was a bug in tcgc, fixed via https://github.com/Azure/typespec-azure/pull/788 and in the dev.3 version of the emitter.

Alancere commented 4 weeks ago

Hi @jhendrixMSFT, the OperationsClient mentioned above has an additional subscriptionID attribute, which seems to have not been fixed

"@azure-tools/typespec-client-generator-core": "0.42.3"
jhendrixMSFT commented 4 weeks ago

Can you also please clarify what the issue is? I verified the original issue was fixed last week.

jhendrixMSFT commented 4 weeks ago

Ok I understand the issue now, we shouldn't be propagating subscriptionID to this specific client.

The difference in OperationListResult vs PagedOperation, this comes from the tsp authoring. Maybe it can be fixed with some renaming in client.tsp.