Use spread over Object.assign

[bogas04]

What: Object.assign is being replaced by Object spread syntax.

Why: Object spread doesn't require you to also include polyfill for Object.assign, hence it's better for users trying it out on older browsers.

Checklist:

• Documentation
• Tests
• Code complete

[codecov-io]

Codecov Report

Merging #193 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   89.13%   89.13%           
=======================================
  Files          22       22           
  Lines         847      847           
  Branches      219      219           
=======================================
  Hits          755      755           
  Misses         70       70           
  Partials       22       22

Impacted Files	Coverage Δ
src/react/index.js	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 72abd4d...f6e7a90. Read the comment docs.

[thangngoc89]

Object spread doesn't require you to also include polyfill for Object.assign, hence it's better for users trying it out on older browsers

Actually, babel includes Object.assign polyfill for object spread.

[bogas04]

@thangngoc89 it includes it for spread, but if you directly use Object.assign, then the user library will need to add polyfill.

[tkh44]

@thangngoc89 he has a point.

[bogas04]

Thank you so much for accepting the PR ! I feel I didn't quite explain myself well. Either way, kudos to Emotion for being awesome !

[thangngoc89]

Yeah. I didn't think about manually providing Object.assign polyfill

[bogas04]

@thangngoc89 Sorry for the rush, but when can I expect this patch to get released to npm ? Thanks!

[thangngoc89]

@bogas04 No idea. Ask @tkh44 . Current he's focusing on shaping version 7.0 which is a major rewrite of emotion core