[Feature Request] Move/remove third party protos to a separate place instead of root
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 領域
- api, build-system
調査の方向性
現在のルートレベルの google/api と google/protobuf のパスを調査し、api-go で protoc が呼び出されている場所を見つけてください。提案されている移動と include-path の変更を比較し、そのうえで、外部ユーザーが独自のサードパーティ製 proto を使用できる状態を保ちながら、リポジトリの protoc コマンドが引き続き動作することを確認してください。
索引モデルが issue の本文から書いたものです。
説明
Describe the solution you'd like
Other users protoc from the root of this repo. When https://github.com/temporalio/api/commit/108b40ae508505b64f19ccbe2fdbcd304081391b was done, we put google-based proto dependencies at the same path as our own assuming that all users were ok with using these google protos instead of their own (e.g. from https://github.com/protocolbuffers/protobuf/tree/main/src/google/protobuf).
We probably need to either:
- Put the third party references where the
protocis actually run (e.g.api-go) and only embed them here when testing
or:
- Move
google/apitothirdparty/google-api/google/api - Move
google/protobuftothirdparty/google-protobuf/google/protobuf - Add
-I thirdparty/google-apiand-I thirdparty/google-protobufto ourprotoc - Confirm the fact that
thirdpartyis nested under our root is ok and can easily be ignored by mostprotocuse
- 主要言語
- Go
- スター
- 133
- フォーク
- 101
- 平均マージ
- 1日 22時間
- マージ済み PR(30日)
- 11
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
temporalio/api のほかの issue
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
temporalio/api#750 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
temporalio/api#427 · コメント 1 件 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
temporalio/api#421 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
temporalio/api#328 · リアクション 6 件 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
temporalio/api#307 ·
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
prometheus/procfs#872 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
bazel-contrib/rules_go#4726 · コメント 1 件 ·
-
area/auto-scaling area/monitoring area/ops-productivity kind/enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100