Skip to content

fix: halt doChorus, doTremolo, doDistortion when depth/distortion validation fails - #7979

Open
Abhishek-Sonje wants to merge 1 commit into
sugarlabs:masterfrom
Abhishek-Sonje:fix/7978-halt-invalid-chorus-tremolo-distortion
Open

fix: halt doChorus, doTremolo, doDistortion when depth/distortion validation fails#7979
Abhishek-Sonje wants to merge 1 commit into
sugarlabs:masterfrom
Abhishek-Sonje:fix/7978-halt-invalid-chorus-tremolo-distortion

fix: halt execution and reset parameters for out-of-range values in s…

2bcd16c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 11, 2026 in 0s

62.05% (+0.00%) compared to 5462731

View this Pull Request on Codecov

62.05% (+0.00%) compared to 5462731

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.05%. Comparing base (5462731) to head (2bcd16c).
⚠️ Report is 34 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7979   +/-   ##
=======================================
  Coverage   62.05%   62.05%           
=======================================
  Files         187      187           
  Lines       58377    58380    +3     
=======================================
+ Hits        36227    36230    +3     
  Misses      22150    22150           

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.