googleapis / googleapis/google-cloud-cpp

renovate.sh ignores real failures in `generate-libraries.sh`

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

説明

We use `renovate.sh` to update the googleapis SHA. It automatically updates the generated code too:

https://github.com/googleapis/google-cloud-cpp/blob/682e0657e270533675163dca54a722526aaae145/external/googleapis/renovate.sh#L72-L75

The `|| true` is needed because `generate-libraries.sh` is intended as a CI script. It is supposed to fail if the code does not match what would happen if we generated the code, and in this case we always generate new code.

It seems we need a way to tell `generate-libraries.sh` to expect code diffs. Or maybe we should refactor the code such that both `renovate.sh` and `generate-libraries.sh` call some shell functions.

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

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

評価

この issue はまだ評価されていません。

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

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