feat(cpp): Add bazel building and testing infrastructure#2785
feat(cpp): Add bazel building and testing infrastructure#2785hubcio merged 7 commits intoapache:masterfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2785 +/- ##
============================================
- Coverage 67.53% 67.52% -0.01%
Complexity 739 739
============================================
Files 1048 1048
Lines 84103 84103
Branches 60704 60716 +12
============================================
- Hits 56795 56791 -4
+ Misses 24950 24944 -6
- Partials 2358 2368 +10
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Nice work on the infra setup. the CI integration and Bazel config are clean. Two things to fix before merge though: the |
|
@atharvalade thanks for the review! those were some things that had been left over from when I was experimenting with the public interface. I've standardized all of the getter functions and corrected the bazel build file as well. |
|
@hubcio Thanks for the review. I'll address the other two comments after thinking about the best approach |
Which issue does this PR close?
Closes #2762
Rationale
What changed?
Adds bazel build system + stub classes and their tests + ci scripts
Local Execution
AI Usage
If AI tools were used, please answer: