Carlos O'Ryan
Carlos O'Ryan
Consider breaking this bug into smaller bugs (one per service?) to make them easier to handle.
> But from what I know, a merge means the issue has been handled right? Some teams break down larger bugs into smaller issues that can be solved in a...
Maybe read about how to run the formatting scripts: https://github.com/googleapis/google-cloud-cpp/tree/main/doc/contributor I know 35GB feels like a lot, but I stopped worrying about this a long time ago.
Looks like you are ready to start, yes. Worst case we discover that you need some other tool / configuration, but you can add those at that time.
I just re-ran the CLA test for you, all clear!
> fatal: detected dubious ownership in repository at '/workspace' > To add an exception for this directory, call: > git config --global --add safe.directory /workspace Hmmm... I have never seen...
An observation: if we want to hide some targets (e.g. `//google/cloud/spanner:spanner_client_testing`) we may want to mark them as deprecated ahead of time.
Yeah, I am just saying that this will require the usual amount of planning. I am trying to keep the scope of #5726 as small as possible, so I am...
We also need to think of experimental features like GCS+gRPC: is is on a separate library target, maybe we should name it `//experimental:gcs_grpc` or something `//:experimental_gcs_grpc`.
>> Any file or symbol that lives within a directory or namespace containing "internal", "impl", "test", "detail", "benchmark", "sample", or "example", is explicitly not part of our public API. >...