googleapis / googleapis/google-cloud-cpp

Pub/Sub: Remove `PleaseIgnoreThisSimplifiesTestingTheSamples` from streaming pull samples

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

説明

Several of the [Pub/Sub streaming pull samples](https://cloud.google.com/pubsub/docs/pull#c++) have a call that looks like `PleaseIgnoreThisSimplifiesTestingTheSamples()`. As I understand it, this is to help with the continuous tests on the samples. However, C++ is the only language that has this and I think it complicates the sample. We should look for a way to remove this.

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

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

調査の方向性

issue にリンクされている C++ Pub/Sub のストリーミング pull サンプルから始めて、PleaseIgnoreThisSimplifiesTestingTheSamples() を検索します。他の言語のサンプルが継続的なテストをどのように扱っているかを比較し、それらのテストを壊さずに呼び出しを削除する方法を特定します。C++ サンプルでこの helper が不要になり、継続的なテストも引き続き動作すれば完了です。

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

評価

技術スタック
cpp, google-cloud
領域
distributed-systems, testing-qa
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

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

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