From c5b4db2f37f0e3495e50299a1e1efe5b7dbf0124 Mon Sep 17 00:00:00 2001
From: Brian Vaughn <bvaughn@fb.com>
Date: Sat, 1 Sep 2018 08:03:24 -0700
Subject: [PATCH] Cleaned up inline comment about scheduling fork

---
 scripts/rollup/forks.js | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/scripts/rollup/forks.js b/scripts/rollup/forks.js
index f82e79dbb2327..28dbd64528afd 100644
--- a/scripts/rollup/forks.js
+++ b/scripts/rollup/forks.js
@@ -123,13 +123,15 @@ const forks = Object.freeze({
       case UMD_DEV:
       case UMD_PROD:
         if (dependencies.indexOf('react') === -1) {
-          // For CJS bundles, use the shared NPM package.
+          // It's only safe to use this fork for modules that depend on React,
+          // because they read the re-exported API from the SECRET_INTERNALS object.
           return null;
         }
         // Optimization: for UMDs, use the API that is already a part of the React
         // package instead of requiring it to be loaded via a separate <script> tag
         return 'shared/forks/ReactScheduler.umd.js';
       default:
+        // For CJS bundles, use the shared NPM package.
         return null;
     }
   },
@@ -139,13 +141,15 @@ const forks = Object.freeze({
       case UMD_DEV:
       case UMD_PROD:
         if (dependencies.indexOf('react') === -1) {
-          // For CJS bundles, use the shared NPM package.
+          // It's only safe to use this fork for modules that depend on React,
+          // because they read the re-exported API from the SECRET_INTERNALS object.
           return null;
         }
         // Optimization: for UMDs, use the API that is already a part of the React
         // package instead of requiring it to be loaded via a separate <script> tag
         return 'shared/forks/ReactSchedulerTracking.umd.js';
       default:
+        // For CJS bundles, use the shared NPM package.
         return null;
     }
   },
@@ -159,13 +163,15 @@ const forks = Object.freeze({
       case UMD_DEV:
       case UMD_PROD:
         if (dependencies.indexOf('react') === -1) {
-          // For CJS bundles, use the shared NPM package.
+          // It's only safe to use this fork for modules that depend on React,
+          // because they read the re-exported API from the SECRET_INTERNALS object.
           return null;
         }
         // Optimization: for UMDs, use the API that is already a part of the React
         // package instead of requiring it to be loaded via a separate <script> tag
         return 'shared/forks/ReactSchedulerTrackingSubscriptions.umd.js';
       default:
+        // For CJS bundles, use the shared NPM package.
         return null;
     }
   },