Skip to content

Commit 590de44

Browse files
committed
chore: Add todo comments
1 parent 851ec06 commit 590de44

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

rust/stackable-cockpit/src/platform/namespace.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ pub enum Error {
1313

1414
/// Creates a namespace with `name` if needed (not already present in the
1515
/// cluster).
16+
// TODO (@NickLarsenNZ): Take a &str instead of String (to avoid all the cloning)
1617
pub async fn create_if_needed(client: &Client, name: String) -> Result<(), Error> {
1718
client
1819
.create_namespace_if_needed(name)

rust/stackable-cockpit/src/platform/stack/spec.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,7 @@ impl StackSpec {
187187
}
188188

189189
// Next, create the product namespace if needed
190+
// TODO (@NickLarsenNZ): Remove clones (update create_if_needed to take a &str)
190191
namespace::create_if_needed(client, install_parameters.stack_namespace.clone())
191192
.await
192193
.context(CreateNamespaceSnafu {

0 commit comments

Comments
 (0)