Skip to content

Conversation

@ddelgrosso1
Copy link
Contributor

@ddelgrosso1 ddelgrosso1 commented May 8, 2025

This change is Reviewable

@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label May 8, 2025
@codecov
Copy link

codecov bot commented May 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.92%. Comparing base (8539ae8) to head (f2fefd3).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15134      +/-   ##
==========================================
- Coverage   92.93%   92.92%   -0.01%     
==========================================
  Files        2391     2391              
  Lines      214995   214995              
==========================================
- Hits       199795   199792       -3     
- Misses      15200    15203       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ddelgrosso1 ddelgrosso1 marked this pull request as ready for review May 8, 2025 18:34
@ddelgrosso1 ddelgrosso1 requested review from a team as code owners May 8, 2025 18:34
Copy link
Contributor

@bajajneha27 bajajneha27 left a comment

Choose a reason for hiding this comment

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

Do we not write sample tests?

@ddelgrosso1
Copy link
Contributor Author

Do we not write sample tests?

No, doesn't appear so for C++. The samples usually run as part of the integration tests but I disabled it for these as creating an AnywhereCache takes an extremely long time.

@bajajneha27
Copy link
Contributor

Do we not write sample tests?

No, doesn't appear so for C++. The samples usually run as part of the integration tests but I disabled it for these as creating an AnywhereCache takes an extremely long time.

I see. We should also add the new file in google/cloud/storage/examples/CMakeLists.txt and google/cloud/storage/examples/storage_examples.bzl

@ddelgrosso1
Copy link
Contributor Author

Do we not write sample tests?

No, doesn't appear so for C++. The samples usually run as part of the integration tests but I disabled it for these as creating an AnywhereCache takes an extremely long time.

I see. We should also add the new file in google/cloud/storage/examples/CMakeLists.txt and google/cloud/storage/examples/storage_examples.bzl

I don't think we want to do that. If we add storage control samples within storage we would then have to include storage control as a dependency of storage.

@bajajneha27
Copy link
Contributor

Do we not write sample tests?

No, doesn't appear so for C++. The samples usually run as part of the integration tests but I disabled it for these as creating an AnywhereCache takes an extremely long time.

I see. We should also add the new file in google/cloud/storage/examples/CMakeLists.txt and google/cloud/storage/examples/storage_examples.bzl

I don't think we want to do that. If we add storage control samples within storage we would then have to include storage control as a dependency of storage.

Oh yeah, this is storage control. My bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants