[Refactoring] Up code quality

.eslintignore

@@ -0,0 +1 @@
+__tests__

.eslintrc

@@ -5,7 +5,8 @@
     },
     "plugins": [
         "react",
-        "react-native"
+        "react-native",
+        "flowtype"
     ],
     "extends": [
         "eslint:recommended",
@@ -15,6 +16,10 @@
         // overrides
     },
     "globals": {
-        "__DEV__": true
+        "__DEV__": true,
+        // @todo flow
+        "State": true,
+        "Promise": true,
+        "ThunkAction": true
     }
 }

App.js

@@ -2,7 +2,7 @@
 // @flow
 
 import React, { Component } from 'react';
-import { View, Text, AppRegistry } from 'react-native';
+import { AppRegistry } from 'react-native';
 import { Provider } from 'react-redux';
 import { Sentry } from 'react-native-sentry';
 import { before } from 'github-flow-js/Client';

Navigator.js

@@ -1,14 +1,13 @@
 // @author Dmitry Patsura <talk@dmtry.me> https://github.com/ovr
 
 import React from 'react';
-import { View, Text, BackHandler } from 'react-native';
+import { BackHandler } from 'react-native';
 import { connect } from 'react-redux';
 import { SideMenuButton } from 'containers';
 import { addNavigationHelpers, StackNavigator, DrawerNavigator, NavigationActions } from 'react-navigation';
 import I18n from 'utils/i18n';
 
 import {
-    Home,
     FeedScreen,
     FeedSettingsScreen,
     LoginScreen,
@@ -29,7 +28,7 @@ export const HomeScreeDrawer = DrawerNavigator({
 }, {
     drawerWidth: 300,
     drawerPosition: 'left',
-    contentComponent: props => <SideMenu />,
+    contentComponent: () => <SideMenu />,
     initialRouteName: 'Home'
 })
 
@@ -101,7 +100,7 @@ class AppWithNavigationState extends React.Component {
             if (navigation.index === 0) {
                 return false;
             }
-                
+
             dispatch(NavigationActions.back());
 
             return true;

actions/flow-types.js

@@ -42,11 +42,11 @@ export type State = {|
     accountPullRequests: AccountPullRequestsState,
     app: AppState,
     login: LoginState,
-    login: NavigationState,
+    navigation: NavigationState,
     profile: ProfileState,
     profileOrganizations: ProfileRepositoriesState,
     profileRepositories: ProfileOrganizationsState,
-    repository: RepositoryState,
+    repository: RepositoryState
 |};
 
 export type Dispatch = (action: Action | ThunkAction | PromiseAction) => any;

actions/navigation.js

@@ -2,7 +2,7 @@
 // @flow
 
 import { NavigationActions } from 'react-navigation';
-import { NAVIGATION_SET_TITLE, NAVIGATION_HOME, NAVIGATION_LOGIN } from 'constants';
+import { NAVIGATION_SET_TITLE } from 'constants';
 import { setupRepository } from './repository';
 
 // import flow types

containers/ProfileRepositories/ProfileRepositories.js

@@ -2,7 +2,7 @@
 // @flow
 
 import React, { PureComponent } from 'react';
-import { View, Text, StyleSheet, FlatList, RefreshControl } from 'react-native';
+import { View, Text, StyleSheet, FlatList } from 'react-native';
 import { connect } from 'react-redux';
 import { RepositoryRow, RowSeparator } from 'components';
 import { fetchRepositories, fetchMoreRepositories, showRepository } from 'actions';
@@ -14,6 +14,8 @@ import type { RepositoryEntity } from 'github-flow-js';
 import type { ProfileRepositoriesState } from 'reducers/profile-repositories';
 
 type Props = {
+    // @todo
+    navigation: Object,
     state: ProfileRepositoriesState,
     fetchRepositories: typeof fetchRepositories,
     fetchMoreRepositories: typeof fetchMoreRepositories,

containers/SideMenu/SideMenu.js

@@ -2,7 +2,7 @@
 // @flow
 
 import React, { PureComponent } from 'react';
-import { Image, View, StyleSheet, Text, ScrollView, Platform, TouchableOpacity } from 'react-native';
+import { View, StyleSheet, Text, ScrollView, Platform, TouchableOpacity } from 'react-native';
 import { Avatar, OrganizationAvatar, Button } from 'components';
 import { connect } from 'react-redux';
 import { hideSideMenu, logout, showAccount, showAbout } from 'actions';
@@ -21,7 +21,7 @@ type Props = {
 
 class SideMenu extends PureComponent<void, Props, void> {
     render() {
-        const { hideSideMenu, app, logout, showAccount, showAbout } = this.props;
+        const { app, logout, showAccount, showAbout } = this.props;
 
         if (app.user === null) {
             return null;

package.json

@@ -37,6 +37,7 @@
     "babel-jest": "20.0.3",
     "babel-preset-react-native": "2.1.0",
     "eslint": "^4.2.0",
+    "eslint-plugin-flowtype": "^2.35.0",
     "eslint-plugin-react": "^7.1.0",
     "eslint-plugin-react-native": "^2.3.2",
     "flow-bin": "^0.49.1",

reducers/navigation.js

@@ -2,7 +2,7 @@
 // @flow
 
 import { AppNavigator } from '../Navigator';
-import { NAVIGATION_SET_TITLE, NAVIGATION_HOME, NAVIGATION_LOGIN } from 'constants';
+// import { NAVIGATION_SET_TITLE, NAVIGATION_HOME, NAVIGATION_LOGIN } from 'constants';
 
 export type NavigationState = {
     params?: any
@@ -13,7 +13,29 @@ const initialState = AppNavigator.router.getStateForAction(
 );
 
 export default (state: NavigationState = initialState, action: Object): NavigationState => {
-    switch (action.type) {
+    let nextState = AppNavigator.router.getStateForAction(action, state);
+
+    // @todo Fix or ask related question inside react-navigation project, because I cannot explain
+    // how to get route params inside state, not route
+    if (nextState && nextState.index) {
+        const route = nextState.routes[nextState.index];
+
+        if (route.params) {
+            nextState = {
+                ...nextState,
+                params: route.params
+            }
+        } else {
+            nextState = {
+                ...nextState,
+                params: null
+            }
+        }
+    }
+
+    return nextState || state;
+
+    // switch (action.type) {
         // case NAVIGATION_LOGIN:
         //     // Reset whole Stack by new State
         //     return AppNavigator.router.getStateForAction(
@@ -24,27 +46,7 @@ export default (state: NavigationState = initialState, action: Object): Navigati
         //     return AppNavigator.router.getStateForAction(
         //         AppNavigator.router.getActionForPathAndParams('Home')
         //     );
-        default:
-            let nextState = AppNavigator.router.getStateForAction(action, state);
-
-            // @todo Fix or ask related question inside react-navigation project, because I cannot explain
-            // how to get route params inside state, not route
-            if (nextState && nextState.index) {
-                const route = nextState.routes[nextState.index];
-
-                if (route.params) {
-                    nextState = {
-                        ...nextState,
-                        params: route.params
-                    }
-                } else {
-                    nextState = {
-                        ...nextState,
-                        params: null
-                    }
-                }
-            }
-
-            return nextState || state;
-    }
+        // default:
+    //
+    // }
 }

utils/filter-concat.js

@@ -1,6 +1,6 @@
 // @flow
 
-import { last } from 'lodash';
+// import { last } from 'lodash';
 
 /**
  * Why is it needed?

utils/team.js

@@ -1,9 +1,6 @@
 // @author Dmitry Patsura <talk@dmtry.me> https://github.com/ovr
 // @flow
 
-// import flow types
-import type { UserEntity } from 'github-flow-js';
-
 export const team = [
     {
         login: "ovr",