googleapis / googleapis/google-cloud-python

Move logic for explicit routing headers to google-api-core

未关闭
#16,272 1 条评论 0 个 reaction 已指派 1 人 已被 @ohmayr 认领 在 GitHub 查看
type: cleanup
主要语言
Python
星标
5.4k
派生
1.8k
平均合并
3 天 4 小时
30 天内合并 PR
122

描述

See feedback in https://github.com/googleapis/gapic-generator-python/pull/2131/files#r1751065571 from review of PR https://github.com/googleapis/gapic-generator-python/pull/2131 which calls out that the logic for explicit routing headers which exists in `_client_macros.j2` could be moved to `google-api-core`, similar to how we have `gapic_v1.routing_header.to_grpc_metadata`

This logic:
https://github.com/googleapis/gapic-generator-python/blob/4c5de8791795f8101f6ec66f80b8a8e5e9a21822/gapic/templates/%25namespace/%25name_%25version/%25sub/services/%25service/_client_macros.j2#L150-L164

Could be refactored into a helper function in `google-api-core` similar to `gapic_v1.routing_header.to_grpc_metadata`

https://github.com/googleapis/gapic-generator-python/blob/4c5de8791795f8101f6ec66f80b8a8e5e9a21822/gapic/templates/%25namespace/%25name_%25version/%25sub/services/%25service/_client_macros.j2#L167-L170

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。