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

Prefer use of Seq constructs for intermediate values. #19

Merged
merged 1 commit into from
Apr 26, 2024
Merged

Conversation

zoj613
Copy link
Owner

@zoj613 zoj613 commented Apr 26, 2024

This commit prefers usage of Seq constructs to avoid creating intermediate lists used for iteration. This results in much more concise code.

This commit prefers usage of Seq constructs to avoid
creating intermediate lists used for iteration. This
results in much more concise code.
Copy link

codecov bot commented Apr 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (ad4f710) to head (4de601f).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines          208       204    -4     
=========================================
- Hits           208       204    -4     

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

@zoj613 zoj613 merged commit ce9d168 into main Apr 26, 2024
6 checks passed
@zoj613 zoj613 deleted the gen32 branch April 26, 2024 13:24
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