Honor the bikeshed by converting public field to getter

vercel · Apr 11, 2023 · d131153 · d131153
1 parent bb54d89
commit d131153
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/crates/turborepo-lib/src/commands/mod.rs b/crates/turborepo-lib/src/commands/mod.rs
@@ -27,7 +27,7 @@ pub struct CommandBase {
     user_config: OnceCell<UserConfig>,
     repo_config: OnceCell<RepoConfig>,
     client_config: OnceCell<ClientConfig>,
-    pub args: Args,
+    args: Args,
     version: &'static str,
 }
 
@@ -128,6 +128,10 @@ impl CommandBase {
         Ok(self.client_config.get().unwrap())
     }
 
+    pub fn args(&self) -> &Args {
+        &self.args
+    }
+
     pub fn api_client(&mut self) -> Result<APIClient> {
         let repo_config = self.repo_config()?;
         let client_config = self.client_config()?;

diff --git a/crates/turborepo-lib/src/execution_state.rs b/crates/turborepo-lib/src/execution_state.rs
@@ -32,7 +32,7 @@ impl<'a> TryFrom<&'a CommandBase> for ExecutionState<'a> {
 
         Ok(ExecutionState {
             remote_config,
-            cli_args: &base.args,
+            cli_args: base.args(),
         })
     }
 }