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

Remove List from src/core #336

Open
1 of 5 tasks
evilsoft opened this issue Oct 27, 2018 · 0 comments
Open
1 of 5 tasks

Remove List from src/core #336

evilsoft opened this issue Oct 27, 2018 · 0 comments

Comments

@evilsoft
Copy link
Owner

evilsoft commented Oct 27, 2018

Description/Location

location: 'src/core/List.js'
description:
The only reason List is in core is for some specs and use in src/Pair/toPairs. It actually makes more sense to have toPairs return an Array of Pairs. With that done

Task List

  • Alter toPairs to return Array instead List
  • Remove/Replace specs using List inside of src/Async/Async.spec.js
  • Remove/Replace specs using List inside of src/Maybe/find.spec.js
  • Move implementation from src/core/List.js and src/core/List.spec.js to src/List/index.js and src/List/List.spec.js
  • Update the documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants