Conversation
The stem EngineBuffer tests use it as std::array's compile-time size. Define it in the support header so enginebuffertest.cpp can use the bound without an out-of-translation-unit definition.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Coverage Report for CI Build 34773419030Warning No base build found for commit Coverage: 30.357%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
2 participants
Publish
fix-signalpath-test-boundas one component PR in the corrected PR23 forest.Topology: parent
baseat6f831813c933d79070600c4b0d47f3318340b0d7(parent_remote_branch: pr23/forest/base). Child tip99b4c77d809053f3be72da0457662b622d067e1e; publish as0cwa:pr23/forest/fix-signalpath-test-bound.Class
small-fix; direct scope (2):src/test/signalpathtest.cpp,src/test/signalpathtest.h.Commits:
99b4c77d809053f3be72da0457662b622d067e1e— Make process buffer size a constexpr test bound.Source anchors:
pr23/fix/signalpath-test-bound,99b4c77d809053f3be72da0457662b622d067e1e.Manifest validation only; CI/GitHub status is not inferred. Known limitations remain as documented in the manifest.