Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/refactor stream #207

Merged
merged 6 commits into from
Feb 14, 2024
Merged

Feature/refactor stream #207

merged 6 commits into from
Feb 14, 2024

Conversation

vitoke
Copy link
Contributor

@vitoke vitoke commented Feb 14, 2024

The @rimbu/stream package has been refactored with the following main differences:

  • filter... and find methods now infer types when the predicate is a type guard
  • More Reducers and AsyncReducers added like partition, groupBy, and race.
  • Combining reducers has been made easier by extending the reduce... methods to accept reducer shapes.
  • Reducers and AsyncReducers can now be halted in the initialisation phase
  • Reducer and AsyncReducers now always require a lazy initialisation value
  • More Transformers and AsyncTransformers have been added like flatMap, flatZip, and filter.
  • Stream and AsyncStream have been extended with convenience methods based on the new reducers.
  • Transformers and convenience methods have been added like splitOn, splitOnSlice.

Copy link

codecov bot commented Feb 14, 2024

Codecov Report

Attention: 152 lines in your changes are missing coverage. Please review.

Comparison is base (3e41aee) 96.91% compared to head (1ad227b) 97.30%.
Report is 3 commits behind head on main.

Files Patch % Lines
...es/stream/src/async-custom/async-stream-custom.mts 90.31% 55 Missing ⚠️
packages/stream/src/custom/stream-custom.mts 91.50% 52 Missing ⚠️
packages/stream/src/async/async-reducer.mts 95.97% 33 Missing ⚠️
...ream/src/async-custom/async-fast-iterator-base.mts 92.92% 7 Missing ⚠️
packages/stream/src/async/async-transformer.mts 98.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #207      +/-   ##
==========================================
+ Coverage   96.91%   97.30%   +0.38%     
==========================================
  Files         321      321              
  Lines       56319    57636    +1317     
  Branches     6587     6807     +220     
==========================================
+ Hits        54583    56083    +1500     
+ Misses       1724     1541     -183     
  Partials       12       12              

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

@vitoke vitoke merged commit 6d50aa8 into main Feb 14, 2024
4 of 5 checks passed
@vitoke vitoke deleted the feature/refactor-stream branch February 14, 2024 07:45
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.

1 participant