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

Allow top level generator types for actions #350

Closed
wants to merge 4 commits into from

Conversation

nhunzaker
Copy link
Contributor

This commit adds a short-hand for generator actions. For example:

let count = n => n

function * range (repo, start, end) {
  while (start <= end)
    yield repo.push(count, start++)
  }
}

repo.push(range, 1, 10) // 1,2,3,4,5,6,7,8,9,10

Related to #346

@nhunzaker nhunzaker force-pushed the react-router-revise branch from d7c73b6 to 6e24813 Compare June 19, 2017 13:24
@nhunzaker nhunzaker force-pushed the action-generator-revise branch from 00bf4e8 to c4239e2 Compare June 19, 2017 13:28
@codecov-io
Copy link

codecov-io commented Jun 19, 2017

Codecov Report

Merging #350 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #350   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          25     25           
  Lines         858    861    +3     
=====================================
+ Hits          858    861    +3
Impacted Files Coverage Δ
src/coroutine.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66d2705...50ce69e. Read the comment docs.

@nhunzaker nhunzaker force-pushed the action-generator-revise branch 2 times, most recently from ee9e9db to 94c7e17 Compare June 19, 2017 14:49
@nhunzaker nhunzaker force-pushed the react-router-revise branch from 6e24813 to 76a5899 Compare July 3, 2017 13:39
@nhunzaker nhunzaker changed the base branch from react-router-revise to master July 3, 2017 13:48
@nhunzaker nhunzaker force-pushed the action-generator-revise branch 2 times, most recently from bb3e4c4 to eb4a27e Compare July 6, 2017 12:06
This commit adds a short-hand for generator actions. For example:

```javascript
let count = n => n

function * range (repo, start, end) {
  while (start <= end)
    yield repo.push(count, start++)
  }
}

repo.push(range, 1, 10) // 1,2,3,4,5,6,7,8,9,10
```
@nhunzaker nhunzaker force-pushed the action-generator-revise branch from eb4a27e to 50ce69e Compare July 6, 2017 12:07
@nhunzaker
Copy link
Contributor Author

I'm going to leave this alone for the time being. I want to get some planning in place first.

@nhunzaker nhunzaker closed this Jul 31, 2017
@nhunzaker nhunzaker deleted the action-generator-revise branch September 26, 2017 19:10
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.

2 participants