From 136dce67cec749dce5989ea29e88359ef79a0045 Mon Sep 17 00:00:00 2001 From: David Sherret Date: Fri, 14 Apr 2023 16:22:33 -0400 Subject: refactor: break up `ProcState` (#18707) 1. Breaks up functionality within `ProcState` into several other structs to break out the responsibilities (`ProcState` is only a data struct now). 2. Moves towards being able to inject dependencies more easily and have functionality only require what it needs. 3. Exposes `Arc` around the "service structs" instead of it being embedded within them. The idea behind embedding them was to reduce the verbosity of needing to pass around `Arc<...>`, but I don't think it was exactly working and as we move more of these structs to be more injectable I don't think the extra verbosity will be a big deal. --- cli/tools/installer.rs | 1 + 1 file changed, 1 insertion(+) (limited to 'cli/tools/installer.rs') diff --git a/cli/tools/installer.rs b/cli/tools/installer.rs index ec397f78a..461bb1a50 100644 --- a/cli/tools/installer.rs +++ b/cli/tools/installer.rs @@ -235,6 +235,7 @@ pub async fn install_command( // ensure the module is cached ProcState::from_flags(flags.clone()) .await? + .module_load_preparer .load_and_type_check_files(&[install_flags.module_url.clone()]) .await?; -- cgit v1.2.3