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.

Sign up for GitHub

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

Jump to bottom

scan{l,r}1 #234

Closed
argantos opened this issue Dec 3, 2019 · 1 comment · Fixed by #276
Closed

scan{l,r}1 #234

argantos opened this issue Dec 3, 2019 · 1 comment · Fixed by #276
Assignees
@vrom911
Labels
reexport Reexport something new
Milestone
v0.7.0.0: Refiner

Comments

@argantos
Copy link

argantos commented Dec 3, 2019

Why are scanl1 and scanr1 missing? They are not partial.
@chshersh chshersh added the reexport Reexport something new label Dec 3, 2019
@chshersh
Copy link
Contributor

chshersh commented Dec 3, 2019

@argantos No particular reason. We happen never to use scan{l,r}[1] so these functions were not considered for addition at first place. I don't mind adding scanl1 and scanr1 to relude as well since these functions are already reexported by relude. Maybe we can even add scanl' as a more efficient version of scanl.

@chshersh chshersh added this to the v0.7.0.0: Refiner milestone Feb 25, 2020
@vrom911 vrom911 self-assigned this May 11, 2020
vrom911 added a commit that referenced this issue May 11, 2020
@vrom911
[#234] scan{l,r}1
6610592 
Resolves #234
@vrom911 vrom911 mentioned this issue May 11, 2020
Merged
vrom911 added a commit that referenced this issue May 11, 2020
@vrom911
[#234] scan{l,r}1
b3000f8 
Resolves #234
chshersh pushed a commit that referenced this issue May 11, 2020
@vrom911
[#234] scan{l,r}1 (#276)
2eb6809 
Resolves #234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vrom911 vrom911

Labels
reexport Reexport something new
Projects
None yet
Milestone
v0.7.0.0: Refiner
Development

Successfully merging a pull request may close this issue.

[#234] scan{l,r}1 kowainik/relude
3 participants
@chshersh @vrom911 @argantos