googleapis / googleapis/google-cloud-cpp

Move microgenerator invocations to Bazel rules

Đang mở
#14,448 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
type: cleanup
Ngôn ngữ chính
C++
Star
659
Fork
462
Merge trung bình
1 ngày 2 giờ
Pull request đã merge (30 ngày)
89

Mô tả

We use scripts to invoke the microgenerator via Bazel, for example:

https://github.com/googleapis/google-cloud-cpp/blob/34c4b4c3f61f716cb993c56dfd72486f427f1073/ci/cloudbuild/builds/generate-libraries.sh#L26-L33

It seems that kind of code really belongs in some kind of `genrule()` or a custom rule. There are many hard-coded magic strings in that call that can be converted to [`Label()`](https://bazel.build/rules/lib/builtins/Label#Label). That returns a `Label` object with all kinds of goodies: https://bazel.build/rules/lib/builtins/Label.html

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

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

Hướng nghiên cứu

Bắt đầu với ci/cloudbuild/builds/generate-libraries.sh quanh các dòng 26-33 và kiểm tra cách nó gọi microgenerator thông qua Bazel. Xác định xem genrule hay custom rule phù hợp; hoàn tất khi script không còn chứa các chi tiết gọi được hard-code và Bazel đảm nhiệm bước sinh đó.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Lĩnh vực
build-system
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

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.