Flow/Lint fix

facebook · May 25, 2018 · a50b70c · a50b70c
1 parent dd4edbc
commit a50b70c
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -81,6 +81,5 @@ module.exports = {
     spyOnDev: true,
     spyOnDevAndProd: true,
     spyOnProd: true,
-    __PROFILER__: true,
   },
 };
diff --git a/packages/shared/ReactFeatureFlags.js b/packages/shared/ReactFeatureFlags.js
@@ -37,7 +37,8 @@ export const warnAboutDeprecatedLifecycles = false;
 export const warnAboutLegacyContextAPI = false;
 
 // Gather advanced timing metrics for Profiler subtrees.
-export const enableProfilerTimer = __DEV__ || __PROFILER__;
+export const enableProfilerTimer =
+  __DEV__ || process.env.REACT_ENABLE_PROFILING === true;
 
 // Fires getDerivedStateFromProps for state *or* props changes
 export const fireGetDerivedStateFromPropsOnStateUpdates = true;

diff --git a/packages/shared/forks/ReactFeatureFlags.native-fabric-oss.js b/packages/shared/forks/ReactFeatureFlags.native-fabric-oss.js
@@ -20,7 +20,8 @@ export const enableSuspense = false;
 export const warnAboutDeprecatedLifecycles = false;
 export const warnAboutLegacyContextAPI = false;
 export const replayFailedUnitOfWorkWithInvokeGuardedCallback = __DEV__;
-export const enableProfilerTimer = __DEV__ || __PROFILER__;
+export const enableProfilerTimer =
+  __DEV__ || process.env.REACT_ENABLE_PROFILING;
 export const fireGetDerivedStateFromPropsOnStateUpdates = true;
 
 // Only used in www builds.

diff --git a/packages/shared/forks/ReactFeatureFlags.native-oss.js b/packages/shared/forks/ReactFeatureFlags.native-oss.js
@@ -20,7 +20,8 @@ export const enableUserTimingAPI = __DEV__;
 export const replayFailedUnitOfWorkWithInvokeGuardedCallback = __DEV__;
 export const warnAboutDeprecatedLifecycles = false;
 export const warnAboutLegacyContextAPI = false;
-export const enableProfilerTimer = __DEV__ || __PROFILER__;
+export const enableProfilerTimer =
+  __DEV__ || process.env.REACT_ENABLE_PROFILING;
 export const fireGetDerivedStateFromPropsOnStateUpdates = true;
 
 // Only used in www builds.

diff --git a/scripts/rollup/build.js b/scripts/rollup/build.js
@@ -290,7 +290,7 @@ function getPlugins(
     // Turn __DEV__ and process.env checks into constants.
     replace({
       __DEV__: isProduction ? 'false' : 'true',
-      __PROFILER__: isProfiling ? 'true' : 'false',
+      'process.env.REACT_ENABLE_PROFILING': isProfiling ? 'true' : 'false',
       'process.env.NODE_ENV': isProduction ? "'production'" : "'development'",
     }),
     // We still need CommonJS for external deps like object-assign.