.bzl スタイルガイド

問題を報告する ソースを表示 ナイトリー · 7.4 .

このページでは、Starlark の基本的なスタイルガイドラインについて説明します。また、マクロとルールに関する情報も記載しています。

Starlark は、ソフトウェアのビルド方法を定義する言語であり、プログラミング言語と構成言語の両方です。

Starlark を使用して、BUILD ファイル、マクロ、ビルドルールを記述します。マクロとルールは基本的にメタ言語であり、BUILD ファイルの作成方法を定義します。BUILD ファイルはシンプルで反復的なファイルです。

すべてのソフトウェアは、書き込まれるよりも多くの回数で読み取られます。これは Starlark の場合に特に当てはまります。エンジニアは BUILD ファイルを読み取って、ターゲットの依存関係とビルドの詳細を把握します。このような読み取りは、多くの場合、合格のとき、急いで行われた場合、または他のタスクの実行と並行して行われます。したがって、ユーザーが BUILD ファイルをすばやく解析して理解できるように、シンプルさと読みやすさが非常に重要です。

ユーザーが BUILD ファイルを開いたときに、そのファイル内のターゲットのリストをすばやく確認したり、その C++ ライブラリのソースのリストを表示したり、その Java バイナリから依存関係を削除したりできます。抽象化レイヤを追加するたびに、ユーザーがこれらのタスクを実行するのが難しくなります。

BUILD ファイルは、さまざまなツールによって分析、更新されます。抽象化を使用している場合、ツールで BUILD ファイルを編集できないことがあります。BUILD ファイルをシンプルに保つことで、より優れたツールを利用できます。コードベースの増大に伴い、ライブラリの更新やクリーンアップのために、多くの BUILD ファイルに変更を加える頻度が高くなります。

一般的なアドバイス

スタイル

Python スタイル

判断に迷う場合は、可能な限り PEP 8 スタイルガイドに従ってください。特に、Python の規則に従って、インデントに 2 つのスペースではなく 4 つのスペースを使用します。

Starlark は Python ではないため、Python スタイルの一部の側面は適用されません。たとえば、PEP 8 では、シングルトンとの比較は is で行うことを推奨しています。これは Starlark の演算子ではありません。

docstring

docstring を使用して、ファイルと関数をドキュメント化します。各 .bzl ファイルの上部にドキュメント コメントを使用し、公開関数ごとにドキュメント コメントを使用します。

ルールとアスペクトを文書化する

ルールとアスペクト、その属性、プロバイダとそのフィールドは、doc 引数を使用してドキュメント化する必要があります。

命名規則

  • 変数名と関数名では、cc_library のように、小文字とアンダースコア([a-z][a-z0-9_]*)で区切られた単語を使用します。
  • 最上位の非公開値は 1 つのアンダースコアで始まります。Bazel では、他のファイルからプライベート値を使用できないようにします。ローカル変数にアンダースコア接頭辞を使用しないでください。

行の長さ

BUILD ファイルと同様に、ラベルは長くできるため、厳密な行長の制限はありません。可能であれば、1 行あたり 79 文字以内に収めるようにしてください(Python のスタイルガイド PEP 8 に沿って)。このガイドラインは厳格に適用すべきではありません。エディタでは 80 列を超える列を表示する必要があります。また、変更を自動化すると頻繁に行が長くなるため、すでに読み取り可能な時間分割行を人間が費やすべきではありません。

キーワード引数

キーワード引数では、等号の周囲にスペースを入れることをおすすめします。

def fct(name, srcs):
    filtered_srcs = my_filter(source = srcs)
    native.cc_library(
        name = name,
        srcs = filtered_srcs,
        testonly = True,
    )

ブール値

ブール値には、10 ではなく、TrueFalse の値を使用する(ルールでブール属性を使用する場合など)。

本番環境のコードでは print() 関数を使用しないでください。この関数はデバッグ専用であり、.bzl ファイルの直接的および間接的なすべてのユーザーにスパム行為を生じさせます。唯一の例外は、print() がデフォルトで無効になっており、ソースの編集によってのみ有効にできる場合に、print() を使用するコードを送信できることです。たとえば、print() のすべての使用が if DEBUG: でガードされ、DEBUGFalse にハードコードされている場合などです。これらのステートメントが、読みやすさへの影響を正当化するのに十分な有用であるかどうかに注意してください。

マクロ

マクロは、読み込みフェーズ中に 1 つ以上のルールをインスタンス化する関数です。一般に、可能な限りマクロではなくルールを使用してください。ユーザーが確認できるビルドグラフは、ビルド中に Bazel が使用するビルドグラフとは異なります。マクロは、Bazel がビルドグラフの分析を行う前に展開されます。

そのため、問題が発生した場合、ユーザーはマクロの実装を理解してビルドの問題をトラブルシューティングする必要があります。また、結果に表示されるターゲットはマクロ展開から取得されるため、bazel query の結果を解釈するのは難しい場合があります。最後に、アスペクトはマクロを認識しないため、アスペクトに依存するツール(IDE など)が失敗する可能性があります。

マクロを安全に使用するには、Bazel CLI または BUILD ファイルで直接参照することを目的とした追加のターゲットを定義します。その場合、それらのターゲットのエンドユーザーのみがそれらについて知る必要があり、マクロによって発生するビルドの問題は、その使用から遠く離れることはありません。

生成されたターゲットを定義するマクロ(CLI で参照されることも、そのマクロによってインスタンス化されないターゲットによって依存されることもないマクロの実装の詳細)の場合は、次のベスト プラクティスに従ってください。

  • マクロは name 引数を受け取り、その名前でターゲットを定義する必要があります。 そのターゲットがそのマクロのメイン ターゲットになります。
  • 生成されたターゲット(マクロで定義された他のすべてのターゲット)は、次の条件を満たしている必要があります。
    • 名前の先頭には <name> または _<name> が付加されます。たとえば、name = '%s_bar' % (name) を使用します。
    • 公開設定が制限されている(//visibility:private)、
    • ワイルドカード ターゲット(:all...:* など)の拡張を回避するために manual タグがある。
  • name は、マクロで定義されたターゲットの名前を導出する場合にのみ使用し、それ以外には使用しないでください。たとえば、マクロ自体によって生成されていない依存関係や入力ファイルを名前を使用して導出しないでください。
  • マクロで作成されたターゲットはすべて、なんらかの方法でメイン ターゲットに結合する必要があります。
  • 従来、マクロを定義する際の最初の引数は name です。
  • マクロ内のパラメータ名は一貫させます。パラメータを属性値としてメイン ターゲットに渡す場合は、名前を同じにします。マクロ パラメータが共通ルール属性(deps など)と同じ目的を果たす場合は、属性と同じように名前を付けます(下記を参照)。
  • マクロを呼び出すときは、キーワード引数のみを使用します。これはルールと一致しており、可読性が大幅に向上します。

エンジニアは、関連するルールの Starlark API が特定のユースケースに不十分な場合、ルールが Bazel 内でネイティブ コードで定義されているか Starlark で定義されているかにかかわらず、マクロを記述することがよくあります。この問題が発生した場合は、API を拡張して目標を達成できるかどうか、ルールの作成者に確認してください。

一般的なルールとして、マクロがルールに似ているほど効果的です。

マクロもご覧ください。

ルール

  • ルール、アスペクト、およびそれらの属性には、小文字の名前(「スネークケース」)を使用する必要があります。
  • ルール名は、依存関係(またはリーフルールの場合はユーザー)の観点から、ルールによって生成されるアーティファクトの主な種類を表す名詞です。これはファイルの接尾辞である必要はありません。たとえば、Python 拡張機能として使用することを目的とした C++ アーティファクトを生成するルールは py_extension という名前にします。ほとんどの言語では、一般的なルールは次のとおりです。
    • *_library - コンパイル単位または「モジュール」。
    • *_binary - 実行可能ファイルまたはデプロイ ユニットを生成するターゲット。
    • *_test - テスト ターゲット。複数のテストを含めることができます。*_test ターゲット内のすべてのテストが同じテーマのバリエーションであることを想定します(単一のライブラリのテストなど)。
    • *_import: .jar やコンパイル中に使用される .dll など、コンパイル済みのアーティファクトをカプセル化するターゲット。
  • 属性には一貫した名前と型を使用します。一般に適用される属性には次のようなものがあります。
    • srcs: label_list。ファイル(通常は人間が作成したソースファイル)を許可します。
    • deps: label_list。通常、ファイル: コンパイル依存関係を許可しません
    • data: label_list。ファイル(テストデータなどのデータファイル)を許可します。
    • runtime_deps: label_list: コンパイルに不要なランタイム依存関係。
  • 動作がわかりにくい属性(特別な置換を含む文字列テンプレートや、特定の要件で呼び出されるツールなど)については、属性の宣言(attr.label_list() など)に doc キーワード引数を使用してドキュメントを提供します。
  • ルールの実装関数は、ほとんどの場合、プライベート関数(先頭にアンダースコアが付いた名前)にする必要があります。一般的なスタイルとして、myrule の実装関数に _myrule_impl という名前を付けます。
  • 明確に定義されたプロバイダ インターフェースを使用して、ルール間で情報を渡します。プロバイダ フィールドを宣言して文書化します。
  • 拡張性を念頭に置いてルールを設計してください。他のルールがルールとやり取りしたり、プロバイダにアクセスしたり、作成したアクションを再利用したりする可能性があることを考慮してください。
  • ルールのパフォーマンス ガイドラインに従います。