From 750a81e3daa495def954125a17d48cd9357fa945 Mon Sep 17 00:00:00 2001
From: Mo Gorhom <gorhom@me.com>
Date: Sat, 16 Jan 2021 13:24:51 +0100
Subject: [PATCH] chore: added no console eslint rule (#199)

---
 .eslintrc.js                                      |  1 +
 example/index.ts                                  |  1 -
 example/src/screens/DummyScreen.tsx               |  1 +
 example/src/screens/advanced/NavigatorExample.tsx |  1 +
 example/src/screens/modal/SimpleExample.tsx       |  2 ++
 example/src/screens/static/BasicExample.tsx       |  2 +-
 example/src/screens/static/BasicExamples.tsx      |  2 ++
 src/components/bottomSheet/BottomSheet.tsx        | 14 +++++++-------
 8 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index a0f4233..cdc0241 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -2,6 +2,7 @@ module.exports = {
   root: true,
   extends: ['@react-native-community', 'prettier'],
   rules: {
+    'no-console': ['error', { allow: ['warn', 'error'] }],
     'prettier/prettier': [
       'error',
       {
diff --git a/example/index.ts b/example/index.ts
index 08c424b..fa9ef26 100644
--- a/example/index.ts
+++ b/example/index.ts
@@ -4,7 +4,6 @@ import { enableScreens } from 'react-native-screens';
 import App from './src/App';
 import { name as appName } from './app.json';
 
-console.disableYellowBox = true;
 enableScreens(true);
 
 AppRegistry.registerComponent(appName, () => App);
diff --git a/example/src/screens/DummyScreen.tsx b/example/src/screens/DummyScreen.tsx
index 6be03c9..07fc67a 100644
--- a/example/src/screens/DummyScreen.tsx
+++ b/example/src/screens/DummyScreen.tsx
@@ -21,6 +21,7 @@ const createDummyScreen = ({
     const { navigate } = useNavigation();
 
     const handleNavigatePress = useCallback(() => {
+      // eslint-disable-next-line no-console
       console.log('navigateTo', nextScreen);
       navigate(nextScreen);
       // eslint-disable-next-line react-hooks/exhaustive-deps
diff --git a/example/src/screens/advanced/NavigatorExample.tsx b/example/src/screens/advanced/NavigatorExample.tsx
index 33f615e..8a10615 100644
--- a/example/src/screens/advanced/NavigatorExample.tsx
+++ b/example/src/screens/advanced/NavigatorExample.tsx
@@ -81,6 +81,7 @@ const NavigatorExample = () => {
 
   // callbacks
   const handleSheetChange = useCallback(index => {
+    // eslint-disable-next-line no-console
     console.log('handleSheetChange', index);
   }, []);
   const handleSnapPress = useCallback(index => {
diff --git a/example/src/screens/modal/SimpleExample.tsx b/example/src/screens/modal/SimpleExample.tsx
index 581479e..66fe941 100644
--- a/example/src/screens/modal/SimpleExample.tsx
+++ b/example/src/screens/modal/SimpleExample.tsx
@@ -10,9 +10,11 @@ const SimpleExample = () => {
 
   // callbacks
   const handleChange = useCallback((index: number) => {
+    // eslint-disable-next-line no-console
     console.log('index', index);
   }, []);
   const handleDismiss = useCallback(() => {
+    // eslint-disable-next-line no-console
     console.log('on dismiss');
   }, []);
   const handleDismissPress = useCallback(() => {
diff --git a/example/src/screens/static/BasicExample.tsx b/example/src/screens/static/BasicExample.tsx
index 96096b3..f4409ed 100644
--- a/example/src/screens/static/BasicExample.tsx
+++ b/example/src/screens/static/BasicExample.tsx
@@ -33,6 +33,7 @@ const BasicExample = () => {
 
   // callbacks
   const handleSheetChanges = useCallback((index: number) => {
+    // eslint-disable-next-line no-console
     console.log('handleSheetChanges', index);
   }, []);
   const handleSnapPress = useCallback(index => {
@@ -46,7 +47,6 @@ const BasicExample = () => {
   }, []);
 
   // renders
-  console.log('BasicExample', 'render');
   return (
     <View style={containerStyle}>
       <Button
diff --git a/example/src/screens/static/BasicExamples.tsx b/example/src/screens/static/BasicExamples.tsx
index 077b1c2..f41e3f6 100644
--- a/example/src/screens/static/BasicExamples.tsx
+++ b/example/src/screens/static/BasicExamples.tsx
@@ -44,11 +44,13 @@ const createExampleScreen = ({ type, count = 20 }: ExampleScreenProps) =>
 
     // callbacks
     const handleSheetChange = useCallback(index => {
+      // eslint-disable-next-line no-console
       console.log('handleSheetChange', index);
     }, []);
 
     const handleSheetAnimate = useCallback(
       (fromIndex: number, toIndex: number) => {
+        // eslint-disable-next-line no-console
         console.log('handleSheetAnimate', `from ${fromIndex} to ${toIndex}`);
       },
       []
diff --git a/src/components/bottomSheet/BottomSheet.tsx b/src/components/bottomSheet/BottomSheet.tsx
index 2820a25..1d6daaf 100644
--- a/src/components/bottomSheet/BottomSheet.tsx
+++ b/src/components/bottomSheet/BottomSheet.tsx
@@ -551,13 +551,13 @@ const BottomSheetComponent = forwardRef<BottomSheet, BottomSheetProps>(
     //#endregion
 
     //#region render
-    console.log(
-      'BottomSheet',
-      'render',
-      snapPoints,
-      sheetHeight,
-      safeHandleHeight
-    );
+    // console.log(
+    //   'BottomSheet',
+    //   'render',
+    //   snapPoints,
+    //   sheetHeight,
+    //   safeHandleHeight
+    // );
     return (
       <BottomSheetProvider value={externalContextVariables}>
         <BottomSheetBackdropContainer