Skip to content

Commit 981ee3d

Browse files
yangdanny97meta-codesync[bot]
authored andcommitted
don't require everything by default
Summary: We're already explicitly checking all the modules we can find w/ Require::Everything when we do transaction.run The other reachable modules are all 3p deps, we don't need to keep all their ASTs and bindings and answers in memory, so the default require level can be Exports 2x speedup, 10x memory usage reduction fixes #1714 Reviewed By: rchen152 Differential Revision: D91727690 fbshipit-source-id: 771f74bac7d8111763eb45d775f7ff049d6c29bf
1 parent e712bd6 commit 981ee3d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pyrefly/lib/commands/report.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,7 @@ impl ReportArgs {
454454
let holder = Forgetter::new(state, false);
455455
let handles = Handles::new(expanded_file_list);
456456
let mut forgetter = Forgetter::new(
457-
holder.as_ref().new_transaction(Require::Everything, None),
457+
holder.as_ref().new_transaction(Require::Exports, None),
458458
true,
459459
);
460460

0 commit comments

Comments
 (0)