chore(pubsub): migrate to std::optional#16231
Conversation
There was a problem hiding this comment.
Code Review
This pull request migrates the codebase from absl::optional to std::optional (along with std::nullopt and std::make_optional) across various files in the google/cloud/storage and google/cloud/pubsub libraries, including tests, benchmarks, and examples. It also removes the absl::optional dependency from the CMake configuration. There are no review comments, and the changes align with modern C++17 standards as outlined in the repository style guide. I have no additional feedback to provide.
a86003b to
9a33136
Compare
9a33136 to
77d0ae7
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16231 +/- ##
==========================================
- Coverage 92.24% 92.24% -0.01%
==========================================
Files 2265 2265
Lines 210126 210126
==========================================
- Hits 193833 193823 -10
- Misses 16293 16303 +10 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Replacing explicit usages of absl::optional with std::optional in the PubSub Client library as part of broader modernization efforts