Skip to content
This repository has been archived by the owner on Sep 4, 2023. It is now read-only.

Set filter and map to use build in funcs #41

Closed
wants to merge 1 commit into from

Conversation

sergiors
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 94.879% when pulling 5bf2008 on sergiors:master into 70efc62 on jackfirth:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 94.879% when pulling 5bf2008 on sergiors:master into 70efc62 on jackfirth:master.

@jackfirth
Copy link
Owner

What's the advantage of reusing the built-in ones?

@sergiors
Copy link
Contributor Author

The project do not be responsible for those implementations. I think the better way to code is when you avoid to code.

@jackfirth
Copy link
Owner

They're one-liners, and the builtin ones aren't an exact match anyway since they return generators instead of lists. I don't think this is worth the effort.

@jackfirth jackfirth closed this Sep 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants