Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use @flow-weak to avoid errors in Flow 0.14 #17

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dbrock
Copy link

@dbrock dbrock commented Sep 6, 2015

I'm attaching the errors below in case someone prefers to actually fix them instead. I didn't know what to do about the "Computed property cannot be assigned with possibly null/undefined value" ones, in particular.

RCTRefreshControl.ios.js:30:23,29: parameter configs
Missing annotation

RCTRefreshControl.ios.js:30:32,39: parameter callback
Missing annotation

RCTRefreshControl.ios.js:31:22,55: call of method findNodeHandle
Error:
RCTRefreshControl.ios.js:39:9,40: assignment of computed property/element
Computed property cannot be assigned with possibly null value
RCTRefreshControl.ios.js:31:22,55: null

RCTRefreshControl.ios.js:31:22,55: call of method findNodeHandle
Error:
RCTRefreshControl.ios.js:39:9,40: assignment of computed property/element
Computed property cannot be assigned with possibly undefined value
RCTRefreshControl.ios.js:31:22,55: undefined

RCTRefreshControl.ios.js:43:27,30: parameter node
Missing annotation

Found 5 errors

Here are the errors in case someone prefers to fix them (I didn't know what to do about the "Computed property cannot be assigned with possibly undefined value" ones):

```
node_modules/react-refresh-control/RCTRefreshControl.ios.js:30:23,29: parameter configs
Missing annotation

node_modules/react-refresh-control/RCTRefreshControl.ios.js:30:32,39: parameter callback
Missing annotation

node_modules/react-refresh-control/RCTRefreshControl.ios.js:31:22,55: call of method findNodeHandle
Error:
node_modules/react-refresh-control/RCTRefreshControl.ios.js:39:9,40: assignment of computed property/element
Computed property cannot be assigned with possibly null value
node_modules/react-refresh-control/RCTRefreshControl.ios.js:31:22,55: null

node_modules/react-refresh-control/RCTRefreshControl.ios.js:31:22,55: call of method findNodeHandle
Error:
node_modules/react-refresh-control/RCTRefreshControl.ios.js:39:9,40: assignment of computed property/element
Computed property cannot be assigned with possibly undefined value
node_modules/react-refresh-control/RCTRefreshControl.ios.js:31:22,55: undefined

node_modules/react-refresh-control/RCTRefreshControl.ios.js:43:27,30: parameter node
Missing annotation

Found 5 errors
```
@dbrock
Copy link
Author

dbrock commented Sep 6, 2015

Oops, I guess @flow-weak isn't even the correct syntax (so the effect is simply to disable Flow)...

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.

1 participant