Skip to content

chore: update CHANGELOG.md for v1.35#7797

Merged
devjgm merged 5 commits into
googleapis:mainfrom
devjgm:release-jan-22
Jan 4, 2022
Merged

chore: update CHANGELOG.md for v1.35#7797
devjgm merged 5 commits into
googleapis:mainfrom
devjgm:release-jan-22

Conversation

@devjgm
Copy link
Copy Markdown
Contributor

@devjgm devjgm commented Jan 4, 2022

This change is Reviewable

@devjgm devjgm requested a review from a team January 4, 2022 15:16
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 690bed60154c4f9ec972131ddcc0332a9daf00f9

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 4, 2022

Codecov Report

Merging #7797 (67a9c85) into main (b289fad) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7797      +/-   ##
==========================================
- Coverage   95.16%   95.15%   -0.01%     
==========================================
  Files        1278     1278              
  Lines      115115   115115              
==========================================
- Hits       109546   109540       -6     
- Misses       5569     5575       +6     
Impacted Files Coverage Δ
google/cloud/bigtable/internal/common_client.h 94.02% <0.00%> (-5.98%) ⬇️
...integration_tests/schema_admin_integration_test.cc 98.91% <0.00%> (-1.09%) ⬇️
google/cloud/pubsub/subscriber_connection_test.cc 97.18% <0.00%> (-0.71%) ⬇️
...cloud/pubsub/internal/subscription_session_test.cc 97.75% <0.00%> (-0.25%) ⬇️
...ud/spanner/integration_tests/client_stress_test.cc 86.18% <0.00%> (+0.65%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b289fad...67a9c85. Read the comment docs.

Comment thread CHANGELOG.md Outdated

### [Bigtable](https://github.com/googleapis/google-cloud-cpp/blob/main/google/cloud/bigtable/README.md)

* feat(pubsublite): stream factories for PSL bidi streams ([#7671](https://github.com/googleapis/google-cloud-cpp/pull/7671))
Copy link
Copy Markdown
Member

@dbolduc dbolduc Jan 4, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be deleted. (Not just because it is a pubsublite change in the bigtable section. The PR was an internal only change)

Also, do we want to add pubsublite to this list now that it has handwritten code?

sections=(
"BigQuery,google/cloud/bigquery"
"Bigtable,google/cloud/bigtable"
"IAM,google/cloud/iam"
"Pub/Sub,google/cloud/pubsub"
"Spanner,google/cloud/spanner"
"Storage,google/cloud/storage"
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the past, we only document changes in libraries once they are GA.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed. I can add pubsublite to the changes.sh script if we want. If I added it now it would include:

### [Pub/Sub Lite](https://github.com/googleapis/google-cloud-cpp/blob/main/google/cloud/pubsublite/README.md)

* feat(pubsublite): stream factories for PSL bidi streams ([#7671](https://github.com/googleapis/google-cloud-cpp/pull/7671))
* feat(generator): add options support to generated clients ([#7683](https://github.com/googleapis/google-cloud-cpp/pull/7683))

If that's useful to include, I can add it.

Comment thread CHANGELOG.md Outdated

### [Bigtable](https://github.com/googleapis/google-cloud-cpp/blob/main/google/cloud/bigtable/README.md)

* feat(pubsublite): stream factories for PSL bidi streams ([#7671](https://github.com/googleapis/google-cloud-cpp/pull/7671))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the pubsublite change, it was just some test refactoring in Bigtable, and a bit confusing to add this in the Bigtable list.

Comment thread CHANGELOG.md Outdated
* doc: declare Secret Manager and Task as GA ([#7792](https://github.com/googleapis/google-cloud-cpp/pull/7792))
* fix: extremely rare race conditions in retry loop ([#7789](https://github.com/googleapis/google-cloud-cpp/pull/7789))
* feat(common): include the request/response type name in the RPC log ([#7782](https://github.com/googleapis/google-cloud-cpp/pull/7782))
* feat(GCS+gRPC): implement parser for BucketMetadata components ([#7766](https://github.com/googleapis/google-cloud-cpp/pull/7766))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes related to GCS+gRPC seem redundant here?

@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 33af84210c9d71da3b4d634bdd9a612a8e60befd

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

Comment thread CHANGELOG.md Outdated
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 32b755bd4d687b8272f8f4f500f0dbc24340734c

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 67a9c85b88463bfcc152e63db072c3922f9453b6

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@devjgm devjgm merged commit eec7062 into googleapis:main Jan 4, 2022
joezqren pushed a commit to joezqren/google-cloud-cpp that referenced this pull request Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants