Don't use the console in prodution

Author: lordofthelake

src/components/Svg/TextPath.tsx

@@ -12,9 +12,10 @@ export default class TextPath extends React.Component<Props> {
   render() {
     if (!this.props.href || !this.props.href.match(idExpReg)) {
       // eslint-disable-next-line no-console
-      console.warn(
-        `Invalid \`href\` prop for \`TextPath\` element, expected a href like \`"#id"\`, but got: "${this.props.href}"`,
-      );
+      if (process.env.NODE_ENV !== 'production')
+        console.warn(
+          `Invalid \`href\` prop for \`TextPath\` element, expected a href like \`"#id"\`, but got: "${this.props.href}"`,
+        );
     }
 
     const { children, ...rest } = this.props;

src/components/Svg/Use.tsx

@@ -22,10 +22,11 @@ export default class Use extends React.Component<Props> {
     const matched = href.match(idExpReg);
 
     if (!href || !matched) {
-      // eslint-disable-next-line no-console
-      console.warn(
-        `Invalid \`href\` prop for \`Use\` element, expected a href like \`"#id"\`, but got: "${href}"`,
-      );
+      if (process.env.NODE_ENV !== 'production')
+        // eslint-disable-next-line no-console
+        console.warn(
+          `Invalid \`href\` prop for \`Use\` element, expected a href like \`"#id"\`, but got: "${href}"`,
+        );
     }
     const { children, ...rest } = this.props;
     return <svg_use {...rest}>{children}</svg_use>;

src/jsonUtils/sketchJson/convertJsonToSketch.ts

@@ -23,7 +23,7 @@ export default function convertJsonToSketch(
   );
 
   if (err.value() !== null) {
-    console.error(err.value());
+    if (process.env.NODE_ENV !== 'production') console.error(err.value());
     throw new Error(err.value());
   }
 

src/jsonUtils/sketchJson/convertSketchToJson.ts

@@ -13,7 +13,7 @@ export default function convertSketchToJson(
   const data = MSJSONDataArchiver.archiveStringWithRootObject_error(imm, err);
 
   if (err.value() !== null) {
-    console.error(err.value());
+    if (process.env.NODE_ENV !== 'production') console.error(err.value());
     throw new Error(err.value());
   }
 

src/render.tsx

@@ -141,8 +141,9 @@ export default async function render(
     const layer = await renderTree(tree, nativeContainer, platformBridge);
     return layer;
   } catch (err) {
-    // eslint-disable-next-line no-console
-    console.error(err);
+    if (process.env.NODE_ENV !== 'production')
+      // eslint-disable-next-line no-console
+      console.error(err);
     const tree = buildTree(<RedBox error={err} />, platformBridge);
     return renderContents(tree, nativeContainer, platformBridge);
   }

src/symbol.tsx

@@ -69,7 +69,8 @@ export const injectSymbols = (
   document?: SketchDocumentData | SketchDocument | WrappedSketchDocument,
 ) => {
   if (!isRunningInSketch()) {
-    console.error('Cannot inject symbols while Sketch is not running');
+    if (process.env.NODE_ENV !== 'production')
+      console.error('Cannot inject symbols while Sketch is not running');
     return;
   }
 

src/utils/processTransform/parseTransformProp.ts

@@ -168,8 +168,9 @@ function appendTransform(transform: string) {
     const [a, c, e, b, d, f] = transformParser.parse(transform);
     pooledMatrix.append(a, b, c, d, e, f);
   } catch (e) {
-    // eslint-disable-next-line no-console
-    console.error(e);
+    if (process.env.NODE_ENV !== 'production')
+      // eslint-disable-next-line no-console
+      console.error(e);
   }
 }