googleapis / googleapis/google-cloud-python

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

Đang mở
#16,272 1 bình luận 0 reaction 1 người được giao Được @ohmayr nhận Xem trên GitHub
type: cleanup
Ngôn ngữ chính
Python
Star
5.4k
Fork
1.8k
Merge trung bình
3 ngày 4 giờ
Pull request đã merge (30 ngày)
122

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.