update comments

dojoengine · Jan 19, 2024 · 2c3d3c2 · 2c3d3c2
1 parent 8454b92
commit 2c3d3c2
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 7 deletions.
diff --git a/crates/sozo/src/commands/auth.rs b/crates/sozo/src/commands/auth.rs
@@ -45,7 +45,6 @@ impl AuthArgs {
         let env_metadata = if config.manifest_path().exists() {
             let ws = scarb::ops::read_workspace(config.manifest_path(), config)?;
 
-            // TODO: Check the updated scarb way to read profile specific values
             dojo_metadata_from_workspace(&ws).and_then(|inner| inner.env().cloned())
         } else {
             None

diff --git a/crates/sozo/src/commands/events.rs b/crates/sozo/src/commands/events.rs
@@ -64,7 +64,6 @@ impl EventsArgs {
         let env_metadata = if config.manifest_path().exists() {
             let ws = scarb::ops::read_workspace(config.manifest_path(), config)?;
 
-            // TODO: Check the updated scarb way to read profile specific values
             dojo_metadata_from_workspace(&ws).and_then(|inner| inner.env().cloned())
         } else {
             None

diff --git a/crates/sozo/src/commands/execute.rs b/crates/sozo/src/commands/execute.rs
@@ -40,7 +40,6 @@ impl ExecuteArgs {
         let env_metadata = if config.manifest_path().exists() {
             let ws = scarb::ops::read_workspace(config.manifest_path(), config)?;
 
-            // TODO: Check the updated scarb way to read profile specific values
             dojo_metadata_from_workspace(&ws).and_then(|inner| inner.env().cloned())
         } else {
             None

diff --git a/crates/sozo/src/commands/migrate.rs b/crates/sozo/src/commands/migrate.rs
@@ -14,6 +14,7 @@ use crate::ops::migration;
 pub struct MigrateArgs {
     #[arg(short, long)]
     #[arg(help = "Perform a dry run and outputs the plan to be executed.")]
+    // TODO: i think dry_run would be more descriptive
     pub plan: bool,
 
     #[arg(long)]
@@ -56,8 +57,6 @@ impl MigrateArgs {
             )?;
         }
 
-        // TODO: Check the updated scarb way to read profile specific values
-
         ws.config().tokio_handle().block_on(migration::execute(&ws, self, target_dir))?;
 
         Ok(())

diff --git a/crates/sozo/src/commands/model.rs b/crates/sozo/src/commands/model.rs
@@ -67,7 +67,6 @@ impl ModelArgs {
         let env_metadata = if config.manifest_path().exists() {
             let ws = scarb::ops::read_workspace(config.manifest_path(), config)?;
 
-            // TODO: Check the updated scarb way to read profile specific values
             dojo_metadata_from_workspace(&ws).and_then(|inner| inner.env().cloned())
         } else {
             None

diff --git a/crates/sozo/src/commands/register.rs b/crates/sozo/src/commands/register.rs
@@ -45,7 +45,6 @@ impl RegisterArgs {
         let env_metadata = if config.manifest_path().exists() {
             let ws = scarb::ops::read_workspace(config.manifest_path(), config)?;
 
-            // TODO: Check the updated scarb way to read profile specific values
             dojo_metadata_from_workspace(&ws).and_then(|inner| inner.env().cloned())
         } else {
             None