Skip to content

Conversation

@johanneslumpe
Copy link
Contributor

Everybody ok with that?

@ooflorent
Copy link
Contributor

LGTM

@emmenko
Copy link
Contributor

emmenko commented Jun 19, 2015

LGTM too.

Refs #140

@johanneslumpe
Copy link
Contributor Author

/cc @gaearon @acdlite

If you guys are fine with this too I'll merge this on

ooflorent added a commit that referenced this pull request Jun 19, 2015
Renamed argument to be more descriptive of its dual nature
@ooflorent ooflorent merged commit 6582910 into master Jun 19, 2015
@ooflorent ooflorent deleted the rename-redux-ctor-argument branch June 19, 2015 08:49
@johanneslumpe
Copy link
Contributor Author

Or like this ;)

@gaearon
Copy link
Contributor

gaearon commented Jun 19, 2015

👍

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.

5 participants