evilsoft / evilsoft/crocks

Remove `List` from `src/core`

Open
#336 0 comments 1 reaction 0 assignees View on GitHub
breaking chore good first issue
Dominant language
JavaScript
Stars
1.6k
Forks
103
PR merge metrics
No merged PRs in 30d

Description

### 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

* [x] 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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.