Apply suggestions from code review

Co-authored-by: Greg Brimble <[email protected]>
cloudflare · Oct 24, 2024 · 1dfe8b0 · 1dfe8b0
1 parent 650dfd7
commit 1dfe8b0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.changeset/chilled-apes-repeat.md b/.changeset/chilled-apes-repeat.md
@@ -2,6 +2,6 @@
 "wrangler": patch
 ---
 
-fix: synchronize observability settings during versions deploy
+fix: synchronize observability settings during `wrangler versions deploy`
 
-When running `wrangler versions deploy`, wrangler will now update observability settings in addition to logpush and `tail_consumers`. Unlike `wrangler deploy`, it will not disable observability when observability is undefined in wrangler.toml
+When running `wrangler versions deploy`, Wrangler will now update `observability` settings in addition to `logpush` and `tail_consumers`. Unlike `wrangler deploy`, it will not disable observability when `observability` is undefined in `wrangler.toml`.
diff --git a/packages/wrangler/src/versions/deploy.ts b/packages/wrangler/src/versions/deploy.ts
@@ -588,7 +588,7 @@ async function maybePatchSettings(
 		},
 	});
 
-	let observability = [];
+	const observability = [];
 	if (patchedSettings.observability) {
 		observability.push(`enabled: ${patchedSettings.observability.enabled}`);
 		if (patchedSettings.observability.head_sampling_rate) {