googleapis / googleapis/google-cloud-cpp

Move microgenerator invocations to Bazel rules

オープン
#14,448 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
type: cleanup
主要言語
C++
スター
659
フォーク
462
平均マージ
1日 2時間
マージ済み PR(30日)
89

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

ci/cloudbuild/builds/generate-libraries.sh の26〜33行付近から始め、Bazel を介して microgenerator をどのように呼び出しているかを調べます。genrule とカスタムルールのどちらが適しているかを判断します。完了とは、スクリプトにハードコードされた呼び出しの詳細がなくなり、Bazel がその生成ステップを管理することです。

索引モデルが issue の本文から書いたものです。

評価

領域
build-system
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。