Skip to content

Conversation

chrishunt
Copy link
Contributor

In response to #8 (which can still be merged later if we want)

This does not change the current behavior at all, it just makes the behavior clear in the tests since it's untested. The second commit is a refactor.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4a24573 on chrishunt:chrishunt/delete-nils-from-arrays into 3067a80 on steveklabnik:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4a24573 on chrishunt:chrishunt/delete-nils-from-arrays into 3067a80 on steveklabnik:master.

@chrishunt
Copy link
Contributor Author

Closing this, it's super old. Re-open later if you want ❤️

@chrishunt chrishunt closed this Aug 6, 2013
@steveklabnik
Copy link
Owner

Okay!

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.

3 participants