fix: Removed all console.log statements

These were just debug messages that were cluttering the console output in development mode. fix #45
nareshbhatia · Oct 18, 2018 · 86baa94 · 86baa94
1 parent b16494e
commit 86baa94
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 32 deletions.
diff --git a/src/adapters/history-adapter.ts b/src/adapters/history-adapter.ts
@@ -26,11 +26,11 @@ export class HistoryAdapter {
     }
 
     goToLocation = (location: Location): Promise<RouterState> => {
-        if (process.env.NODE_ENV === 'development') {
-            console.log(
-                `HistoryAdapter.goToLocation(${JSON.stringify(location)})`
-            );
-        }
+        // if (process.env.NODE_ENV === 'development') {
+        //     console.log(
+        //         `HistoryAdapter.goToLocation(${JSON.stringify(location)})`
+        //     );
+        // }
 
         // Find the matching route
         const routes = this.routerStore.routes;
@@ -74,12 +74,12 @@ export class HistoryAdapter {
                 );
                 if (currentUrl !== routerStateUrl) {
                     this.history.push(routerStateUrl);
-                    if (process.env.NODE_ENV === 'development') {
-                        console.log(
-                            `HistoryAdapter: history.push(${routerStateUrl}),`,
-                            `history.length=${this.history.length}`
-                        );
-                    }
+                    // if (process.env.NODE_ENV === 'development') {
+                    //     console.log(
+                    //         `HistoryAdapter: history.push(${routerStateUrl}),`,
+                    //         `history.length=${this.history.length}`
+                    //     );
+                    // }
                 }
             }
         );

diff --git a/src/adapters/static-adapter.ts b/src/adapters/static-adapter.ts
@@ -17,12 +17,12 @@ export class StaticAdapter {
     }
 
     goToLocation = (location: Location): Promise<RouterState> => {
-        /* istanbul ignore if */
-        if (process.env.NODE_ENV === 'development') {
-            console.log(
-                `StaticAdapter.goToLocation(${JSON.stringify(location)})`
-            );
-        }
+        // /* istanbul ignore if */
+        // if (process.env.NODE_ENV === 'development') {
+        //     console.log(
+        //         `StaticAdapter.goToLocation(${JSON.stringify(location)})`
+        //     );
+        // }
 
         // Find the matching route
         const routes = this.routerStore.routes;

diff --git a/src/components/router-view.tsx b/src/components/router-view.tsx
@@ -23,9 +23,9 @@ export class RouterView extends React.Component<RouterViewProps, {}> {
             routerStore: { routerState },
             viewMap
         } = this.props;
-        if (process.env.NODE_ENV === 'development') {
-            console.log(`RouterView.render() - ${JSON.stringify(routerState)}`);
-        }
+        // if (process.env.NODE_ENV === 'development') {
+        //     console.log(`RouterView.render() - ${JSON.stringify(routerState)}`);
+        // }
 
         const view = viewMap[routerState.routeName];
         return view ? view : null;

diff --git a/src/router-store.ts b/src/router-store.ts
@@ -192,22 +192,22 @@ export class RouterStore {
             /* istanbul ignore if */
             if (process.env.NODE_ENV === 'development') {
                 const fromStateStr = JSON.stringify(fromState);
-                console.log(
-                    `RouterStore.transition(${fromStateStr}):`,
-                    'states are equal, skipping'
-                );
+                // console.log(
+                //     `RouterStore.transition(${fromStateStr}):`,
+                //     'states are equal, skipping'
+                // );
             }
             return toState;
         }
 
-        /* istanbul ignore if */
-        if (process.env.NODE_ENV === 'development') {
-            const fromStateStr = JSON.stringify(fromState);
-            const toStateStr = JSON.stringify(toState);
-            console.log(
-                `RouterStore.transition(${fromStateStr}, ${toStateStr})`
-            );
-        }
+        // /* istanbul ignore if */
+        // if (process.env.NODE_ENV === 'development') {
+        //     const fromStateStr = JSON.stringify(fromState);
+        //     const toStateStr = JSON.stringify(toState);
+        //     console.log(
+        //         `RouterStore.transition(${fromStateStr}, ${toStateStr})`
+        //     );
+        // }
 
         // Get transition hooks from the two states
         const { beforeExit, onExit } = this.getRoute(fromState.routeName);