Skip to content
This repository was archived by the owner on Jan 16, 2023. It is now read-only.

Conversation

gfx
Copy link
Contributor

@gfx gfx commented Aug 22, 2016

As described in #39, and related to #31.


(Updated)

Finally I have inlined react-utils' map() into the project. It's really easy and coverage safe.

@gfx gfx changed the title Merge react-utils into this project (#31) Merge react-utils into this project Aug 22, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-6.6%) to 92.925% when pulling c3f6003 on gfx:merge_react_utils into 5411aa8 on alexcurtis:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.6%) to 92.925% when pulling c3f6003 on gfx:merge_react_utils into 5411aa8 on alexcurtis:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 98.995% when pulling 1c7215f on gfx:merge_react_utils into 5411aa8 on alexcurtis:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.479% when pulling 4644500 on gfx:merge_react_utils into 5411aa8 on alexcurtis:master.

@alexcurtis
Copy link
Collaborator

@gfx Very nice. Thanks! I'll get this merged sometime today and published to npm.

@gfx
Copy link
Contributor Author

gfx commented Aug 22, 2016

Thanks! Can't wait! 😆

@gfx
Copy link
Contributor Author

gfx commented Sep 1, 2016

Hi. How's it going?

@gfx
Copy link
Contributor Author

gfx commented Sep 12, 2016

Resolved the conflict.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.003%) to 99.479% when pulling 3ba4091 on gfx:merge_react_utils into 9d0cab6 on alexcurtis:master.

@alexcurtis alexcurtis merged commit a2c1d73 into storybook-eol:master Sep 12, 2016
@alexcurtis
Copy link
Collaborator

@gfx sorry its taken so long. This has been merged + published to npm. Thanks for your help! 👍

@gfx gfx deleted the merge_react_utils branch September 13, 2016 00:45
@gfx
Copy link
Contributor Author

gfx commented Sep 13, 2016

No problem. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants