Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add "node_modules" to .gitignore. Clarify how head() works. Add pop() method. #28

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
1 participant

I wanted to use this library, but was pretty confused by how head() works. The tests did not really clarify anything.

So, I added tests to document how head() works and simplified the implementation. Turns out it doesn't affect the list, so I added pop() (which does).

Thanks for a really helpful library!

bobbydavid added some commits Jan 19, 2013

simplify method and improve testing for head()
The head() method was implemented so that it does NOT remove
the top value of the list. This refactor makes that more clear,
and adds tests to document that behavior.

This commit should not change the behavior of head() at all.
add pop() with tests.
The head() method looks at the top value, but does not remove it
from the lazy list. The new method, pop(), will remove the value.

lazy.pop(f) == lazy.head(f); lazy.skip(1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment