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

Safe traverse, fix #86 #87

Merged
merged 2 commits into from Nov 2, 2016
Merged

Safe traverse, fix #86 #87

merged 2 commits into from Nov 2, 2016

Conversation

paf31
Copy link
Contributor

@paf31 paf31 commented Nov 1, 2016

I haven't implemented this for lazy lists, since it was still causing a stack overflow, and I couldn't figure out why.

@garyb
Copy link
Member

@garyb garyb commented Nov 1, 2016

Looks good to me, what's with the non-alphabetical import Data.NonEmpty as NE import though? 😄

@garyb
Copy link
Member

@garyb garyb commented Nov 1, 2016

(Not that it matters particularly - just wondered if it tool applied or something, in which case I have a suggestion for the tool 😉)

@paf31
Copy link
Contributor Author

@paf31 paf31 commented Nov 1, 2016

It must have been psc-ide since I didn't change the order on purpose 😄

@paf31 paf31 merged commit 886d3a7 into master Nov 2, 2016
2 checks passed
@paf31 paf31 deleted the safe-traverse branch Nov 2, 2016
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.

None yet

2 participants