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

refactor: use flatMap instead of reduce #1541

Merged
merged 1 commit into from
Mar 29, 2024
Merged

refactor: use flatMap instead of reduce #1541

merged 1 commit into from
Mar 29, 2024

Conversation

G-Rath
Copy link
Collaborator

@G-Rath G-Rath commented Mar 29, 2024

I realised this while doing #1540 - shaves off a few lines which is nice

@G-Rath G-Rath merged commit 295120e into next Mar 29, 2024
26 checks passed
@G-Rath G-Rath deleted the use-flat-map branch March 29, 2024 20:12
Copy link

🎉 This PR is included in version 28.0.0-next.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@SimenB
Copy link
Member

SimenB commented Mar 29, 2024

@G-Rath
Copy link
Collaborator Author

G-Rath commented Mar 29, 2024

I don't think it's worth it given we don't already use unicorn and this is literally all the ones it could flag 🤷

(+ we've been pretty stable for a while so I don't expect a lot of new code)

Copy link

github-actions bot commented Apr 6, 2024

🎉 This PR is included in version 28.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants