Skip to content

Comments

perf: Reduce the amount of alt code generated#864

Draft
epage wants to merge 2 commits intowinnow-rs:mainfrom
epage:macro
Draft

perf: Reduce the amount of alt code generated#864
epage wants to merge 2 commits intowinnow-rs:mainfrom
epage:macro

Conversation

@epage
Copy link
Collaborator

@epage epage commented Jan 8, 2026

No description provided.

@epage epage marked this pull request as draft January 8, 2026 20:24
@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

❌ Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.18%. Comparing base (4459a77) to head (db9d1b6).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/combinator/branch.rs 83.33% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #864      +/-   ##
==========================================
+ Coverage   47.17%   48.18%   +1.01%     
==========================================
  Files          27       31       +4     
  Lines        3536     3607      +71     
==========================================
+ Hits         1668     1738      +70     
- Misses       1868     1869       +1     

☔ 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.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 20830641103

Details

  • 15 of 18 (83.33%) changed or added relevant lines in 1 file are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+1.0%) to 48.184%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/combinator/branch.rs 15 18 83.33%
Files with Coverage Reduction New Missed Lines %
src/stream/mod.rs 1 45.29%
src/combinator/branch.rs 2 81.58%
Totals Coverage Status
Change from base Build 20621066376: 1.0%
Covered Lines: 1738
Relevant Lines: 3607

💛 - Coveralls

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants