Everyone likes to flow these days by jirikolarik · Pull Request #52 · werein/react

jirikolarik · 2017-10-24T13:08:54Z

https://medium.com/@jirikolarik/comprehensive-flow-with-react-redux-bd515d8d0e50

houndci-bot · 2017-10-24T13:09:08Z

 import { routerMiddleware, connectRouter } from 'connected-react-router';
 // Import all reducers
-import * as reducers from 'reducers';
+import * as reducers from 'reducers/index';


Unable to resolve path to module 'reducers/index' import/no-unresolved
Missing file extension for "reducers/index" import/extensions

houndci-bot · 2017-10-24T13:09:08Z

@@ -1,10 +1,12 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions
Resolve error: package.json not found in path import/no-absolute-path

houndci-bot · 2017-10-24T13:09:08Z

@@ -1,3 +1,5 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions
Resolve error: package.json not found in path import/no-absolute-path
Resolve error: package.json not found in path import/no-named-as-default
Resolve error: package.json not found in path import/no-named-as-default-member

houndci-bot · 2017-10-24T13:09:09Z

+// @flow
+
+import type { Dispatch } from 'actions/index';
+import type { AppState } from './app';


Unable to resolve path to module './app' import/no-unresolved
Missing file extension for "./app" import/extensions

houndci-bot · 2017-10-24T13:09:09Z

@@ -1,3 +1,18 @@
+// @flow
+
+import type { Dispatch } from 'actions/index';


Unable to resolve path to module 'actions/index' import/no-unresolved
Missing file extension for "actions/index" import/extensions

houndci-bot · 2017-10-24T13:09:09Z

@@ -1,3 +1,5 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions
Resolve error: package.json not found in path import/no-absolute-path
Resolve error: package.json not found in path import/no-named-as-default
Resolve error: package.json not found in path import/no-named-as-default-member

houndci-bot · 2017-10-24T13:09:09Z


  render() {
-    if (!this.props.loaded) {
+    const { loaded } = this.props;


'loaded' is missing in props validation react/prop-types

houndci-bot · 2017-10-24T13:09:10Z

  componentDidMount() {
-    this.props.dispatch(loadApp());
-  }
+    const { dispatch } = this.props;


'dispatch' is missing in props validation react/prop-types

houndci-bot · 2017-10-24T13:09:10Z

 import { connect } from 'react-redux';
 import { loadApp } from 'actions/app';
+import type { Dispatch } from 'actions/index';
+import type { State } from 'reducers/index';


Unable to resolve path to module 'reducers/index' import/no-unresolved
Missing file extension for "reducers/index" import/extensions

houndci-bot · 2017-10-24T13:09:10Z

 import React, { Component } from 'react';
 import { connect } from 'react-redux';
 import { loadApp } from 'actions/app';
+import type { Dispatch } from 'actions/index';


Unable to resolve path to module 'actions/index' import/no-unresolved
Missing file extension for "actions/index" import/extensions

houndci-bot · 2017-11-07T12:39:46Z

@@ -1,6 +1,12 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates

houndci-bot · 2017-11-07T12:39:46Z

+
+import type {
+  LoadApp
+} from './app';


Unable to resolve path to module './app' import/no-unresolved
Missing file extension for "./app" import/extensions

houndci-bot · 2017-11-07T12:39:46Z

@@ -0,0 +1,9 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions

houndci-bot · 2017-11-07T12:39:46Z

 import { loadApp } from 'actions/app';
+import type { MapStateToProps } from 'react-redux';
+import type { Dispatch } from 'actions/index';
+import type { State } from 'reducers/index';


Unable to resolve path to module 'reducers/index' import/no-unresolved
Missing file extension for "reducers/index" import/extensions

houndci-bot · 2017-11-07T12:39:46Z

@@ -1,6 +1,12 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates

houndci-bot · 2017-11-07T12:39:46Z

+
+import type {
+  LoadApp
+} from './app';


Unable to resolve path to module './app' import/no-unresolved
Missing file extension for "./app" import/extensions

houndci-bot · 2017-11-07T12:39:47Z

@@ -0,0 +1,9 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions

houndci-bot · 2017-11-07T12:39:47Z

 import { loadApp } from 'actions/app';
+import type { MapStateToProps } from 'react-redux';
+import type { Dispatch } from 'actions/index';
+import type { State } from 'reducers/index';


Unable to resolve path to module 'reducers/index' import/no-unresolved
Missing file extension for "reducers/index" import/extensions

houndci-bot · 2017-11-07T12:44:31Z

 import { connect } from 'react-redux';
 import { loadApp } from 'actions/app';
+import type { MapStateToProps } from 'react-redux';
+import type { Dispatch } from 'actions/index';


Unable to resolve path to module 'actions/index' import/no-unresolved
Missing file extension for "actions/index" import/extensions

houndci-bot · 2017-11-07T12:44:31Z

 import React, { Component } from 'react';
 import { connect } from 'react-redux';
 import { loadApp } from 'actions/app';
+import type { MapStateToProps } from 'react-redux';


Unable to resolve path to module 'react-redux' import/no-unresolved
Missing file extension for "react-redux" import/extensions

houndci-bot · 2017-11-07T12:44:31Z

@@ -1,22 +1,32 @@
+// @flow


Resolve error: package.json not found in path import/no-unresolved
Resolve error: package.json not found in path import/no-duplicates
Resolve error: package.json not found in path import/extensions
Resolve error: package.json not found in path import/no-named-as-default
Resolve error: package.json not found in path import/no-named-as-default-member

everyone likes to flow these days

42ff73a

houndci-bot reviewed Oct 24, 2017

View reviewed changes

Add missing parts

6ead2ce

houndci-bot reviewed Nov 7, 2017

View reviewed changes

Update travis

4da639b

houndci-bot reviewed Nov 7, 2017

View reviewed changes

Jiri Kolarik added 2 commits November 7, 2017 13:54

Fix appveyor

73c523c

Install flowtype first

5d18b38

Navigation Menu

Uh oh!

Everyone likes to flow these days - #52

Everyone likes to flow these days#52
jirikolarik wants to merge 5 commits into
masterfrom
flow

jirikolarik commented Oct 24, 2017 •

edited

Loading

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Oct 24, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

houndci-bot Nov 7, 2017

Labels

2 participants

		@@ -1,3 +1,18 @@
		// @flow

		import type { Dispatch } from 'actions/index';

Uh oh!

Conversation

jirikolarik commented Oct 24, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Labels

2 participants

jirikolarik commented Oct 24, 2017 •

edited

Loading