Skip to content
This repository was archived by the owner on Sep 10, 2022. It is now read-only.

Use $Compose instead Compose#523

Merged
istarkov merged 2 commits intomasterfrom
flow-compose
Sep 21, 2017
Merged

Use $Compose instead Compose#523
istarkov merged 2 commits intomasterfrom
flow-compose

Conversation

@istarkov
Copy link
Copy Markdown
Contributor

Fixes almost all flow typing issues we had!!!!!
Waiting for flow-bin to be updated.

@istarkov istarkov self-assigned this Sep 20, 2017
Repository owner deleted a comment from codecov-io Sep 20, 2017
@@ -64,64 +64,6 @@ declare module 'recompose' {

declare type UnaryFn<A, R> = (a: A) => R

declare type Compose = (<A, B, C, D, E, F, G, H, I>(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ha, nice to see this go, we hit the limit again not long after I submitted the earlier PR to extend it 😅

@codecov-io
Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #523   +/-   ##
=======================================
  Coverage   88.94%   88.94%           
=======================================
  Files          53       53           
  Lines         389      389           
=======================================
  Hits          346      346           
  Misses         43       43

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 3a4d7ec...6124500. Read the comment docs.

@istarkov istarkov merged commit 6064554 into master Sep 21, 2017
@istarkov istarkov deleted the flow-compose branch September 21, 2017 23:39
withProps(props => ({
a: (props.a: string),
// Must $ ExpectError but not
// $ExpectError but not
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If it does throw error, we can remove but not as well.

neighborhood999 added a commit to neighborhood999/recompose that referenced this pull request Oct 25, 2017
* export `createEventHandlerWithConfig()` and add documentation.

* Fix Prettier issues

* Docs: Fix missing comma in
 type description for `createEventHandlerWithConfig()`

* Docs: Add *WithConfig functions to TOC.

* Use $Compose instead Compose (acdlite#523)

* Use $Compose instead Compose

* Update flow-bin dependencies

* Allow React 16 (acdlite#530)

Resolves acdlite#506.

* recompose v0.25.1

* Updated base fiddle link (acdlite#534)

Base fiddle was using a unpkg link pointed at the old (React 15) dist url but without a React version, so broke with the release of React 16. Now points to the correct location for React 16, with 16 specified (shouldn't break this way again).

* Remove createEagerFactory and createEagerElement (acdlite#538)

* recompose v0.26.0

* Fix grammar in API.md (acdlite#542)

* Translate `createEventHandlerWithConfig()`
neighborhood999 added a commit to neighborhood999/recompose that referenced this pull request Nov 29, 2017
* export `createEventHandlerWithConfig()` and add documentation.

* Fix Prettier issues

* Docs: Fix missing comma in
 type description for `createEventHandlerWithConfig()`

* Docs: Add *WithConfig functions to TOC.

* Use $Compose instead Compose (acdlite#523)

* Use $Compose instead Compose

* Update flow-bin dependencies

* Allow React 16 (acdlite#530)

Resolves acdlite#506.

* recompose v0.25.1

* Updated base fiddle link (acdlite#534)

Base fiddle was using a unpkg link pointed at the old (React 15) dist url but without a React version, so broke with the release of React 16. Now points to the correct location for React 16, with 16 specified (shouldn't break this way again).

* Remove createEagerFactory and createEagerElement (acdlite#538)

* recompose v0.26.0

* Fix grammar in API.md (acdlite#542)

* Translate `createEventHandlerWithConfig()`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants