@samitouri / QOS-React-2 / commits / abd7bcd8b2

Flow: remove max_workers setting (#25349)

This was added back in #17880 to make CI pass for an unrelated change. This limits the max worker setting to CI environments as removing the setting completely still seems to break on CircleCI.

Jan Kassens committed Sep 29, 2022 at 14:49 UTC abd7bcd8b2d2e4688a5427278003dd6e1b255c3b
2 files changed +6 -1
scripts/flow/config/flowconfig
+1 -1
@@ -39,7 +39,7 @@
39 untyped-type-import=error
40
41 [options]
42 -server.max_workers=4
42 +%CI_MAX_WORKERS%
43 esproposal.class_static_fields=enable
44 esproposal.class_instance_fields=enable
45 esproposal.optional_chaining=enable
scripts/flow/createFlowConfigs.js
+5
@@ -45,6 +45,11 @@ function writeConfig(renderer, rendererInfo, isServerSupported) {
45 });
46
47 const config = configTemplate
48 + .replace(
49 + '%CI_MAX_WORKERS%\n',
50 + // On CI, we seem to need to limit workers.
51 + process.env.CI ? 'server.max_workers=4\n' : '',
52 + )
53 .replace(
54 '%REACT_RENDERER_FLOW_OPTIONS%',
55 `