Skip to content

Conversation

@jorgerangel-msft
Copy link
Contributor

@jorgerangel-msft jorgerangel-msft commented Jan 23, 2026

This PR adds support for another multiservice client scenario where operations exist directly on the root client and not sub-clients.

contributes to: #9032

@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Jan 23, 2026
@jorgerangel-msft jorgerangel-msft force-pushed the multiservice-combined-client branch from a18a014 to 7ee38f7 Compare January 23, 2026 23:22
@github-actions
Copy link
Contributor

No changes needing a change description found.

@jorgerangel-msft jorgerangel-msft force-pushed the multiservice-combined-client branch from 7ee38f7 to 49b83b1 Compare January 23, 2026 23:31
@jorgerangel-msft jorgerangel-msft changed the title Add Support for Multiservice Combined Client Add Support for Multiservice Root Client Jan 23, 2026
@jorgerangel-msft jorgerangel-msft marked this pull request as ready for review January 24, 2026 00:08
@jorgerangel-msft jorgerangel-msft changed the title Add Support for Multiservice Root Client Add Support for Multiservice Root DataPlane Client Jan 24, 2026
}

return _apiVersionFields.FirstOrDefault(
f => string.Equals(f.ServiceNamespace, serviceNamespace, StringComparison.OrdinalIgnoreCase))?.Field;
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: use == as it is equivalent

}

private IReadOnlyList<FieldProvider> BuildAdditionalClientFields()
private List<FieldProvider> BuildAdditionalClientFields()
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change used somewhere?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants