-
Notifications
You must be signed in to change notification settings - Fork 115
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
Add builtins.split #154
Add builtins.split #154
Conversation
The existing commit fails to build on my machine.
Finishes issue haskell-nix#113
I'd like to avoid a dependency on |
Actually, never mind that, they don't have an implementation of |
@mightybyte Is this close to being ready? |
The split implementation is ready but it's still not passing yet because there's one test that includes an indented string and that's not working yet. |
Ah, since there's already an issue for that, we shouldn't hold up your split work. The scope of the PR just needs to be narrowed down to just the changes needed for split. |
Ok, then this should be ready. It still won't pass the test case unless you comment out the one with the indented text. |
Merged by hand. |
No description provided.