Skip to content

Conversation

@agraboso
Copy link
Contributor

No description provided.

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'd rather not use ES7 experimental code in examples to avoid confusing beginners.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are several other instances of object spread notation in the README, both in the text and in the examples. If you want, I can revert them all back to Object.assign.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, in fact there were only the two that I introduced and two more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I'd prefer to use assign everywhere.

gaearon added a commit that referenced this pull request Aug 21, 2015
Fix some typos and inconsistencies in documentation
@gaearon gaearon merged commit 1a0239a into reduxjs:master Aug 21, 2015
@gaearon
Copy link
Contributor

gaearon commented Aug 21, 2015

Thanks.

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