Skip to content

Commit 40c7110

Browse files
committed
Remove pointless comments
1 parent d893975 commit 40c7110

File tree

2 files changed

+1
-13
lines changed

2 files changed

+1
-13
lines changed

src/redis_release/bht/conversation_behaviours.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ def update(self) -> Status:
5858
if first_word in command_map:
5959
self.state.command = command_map[first_word]
6060

61-
# Always return SUCCESS
6261
return Status.SUCCESS
6362

6463

@@ -141,7 +140,6 @@ def update(self) -> Status:
141140

142141
try:
143142
assert self.llm is not None
144-
# Call LLM with structured outputs
145143
response = self.llm.responses.parse(
146144
model="gpt-4o-2024-08-06",
147145
input=[
@@ -156,7 +154,6 @@ def update(self) -> Status:
156154

157155
self.logger.debug(f"LLM response: {response}")
158156

159-
# Extract parsed result from structured output
160157
result = cast(CommandDetectionResult, response.output_parsed)
161158
if not result:
162159
self.feedback_message = "LLM returned empty response"
@@ -188,7 +185,6 @@ def update(self) -> Status:
188185
try:
189186
self.state.command = command
190187

191-
# If help command, set reply
192188
if command == Command.HELP:
193189
self.state.replies.append(
194190
reply or "How can I help you with Redis release automation?"
@@ -211,7 +207,6 @@ def update(self) -> Status:
211207
elif command == Command.RELEASE:
212208
if result.release_args:
213209

214-
# Create ReleaseArgs with converted types
215210
self.state.release_args = ReleaseArgs(
216211
release_tag=result.release_args.release_tag,
217212
force_rebuild=result.release_args.force_rebuild,
@@ -268,7 +263,6 @@ def __init__(
268263
def update(self) -> Status:
269264
self.logger.debug("RunStatusCommand - loading and posting release status")
270265

271-
# Check if we have release args
272266
if not self.state.release_args:
273267
self.feedback_message = "No release args available"
274268
return Status.FAILURE
@@ -277,10 +271,8 @@ def update(self) -> Status:
277271
self.feedback_message = "Command is not STATUS"
278272
return Status.FAILURE
279273

280-
# Mark command as started
281274
self.state.command_started = True
282275

283-
# Get release args
284276
release_args = self.state.release_args
285277

286278
self.logger.info(f"Loading status for tag {release_args.release_tag}")
@@ -355,7 +347,6 @@ def __init__(
355347
def update(self) -> Status:
356348
self.logger.debug("RunCommand - starting release execution")
357349

358-
# Check if we have release args
359350
if not self.state.release_args:
360351
self.feedback_message = "No release args available"
361352
return Status.FAILURE
@@ -364,10 +355,8 @@ def update(self) -> Status:
364355
self.feedback_message = "Command is not RELEASE"
365356
return Status.FAILURE
366357

367-
# Mark command as started
368358
self.state.command_started = True
369359

370-
# Get release args
371360
release_args = self.state.release_args
372361

373362
# Check authorization
@@ -436,7 +425,6 @@ def run_release_in_thread() -> None:
436425
release_thread.start()
437426
self.logger.info(f"Started release thread for tag {release_args.release_tag}")
438427

439-
# Set reply to inform user
440428
self.state.replies.append(
441429
f"Starting release for tag `{release_args.release_tag}`... "
442430
"I'll post updates as the release progresses."

src/redis_release/bht/conversation_tree.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
logger = logging.getLogger(__name__)
3030

3131

32+
# Use redis-release conversation-print command to visualize the tree
3233
def create_conversation_root_node(
3334
input: InboxMessage,
3435
config: Config,
@@ -116,7 +117,6 @@ def initialize_conversation_tree(
116117
reply_queue: Optional[SyncQueue] = None,
117118
) -> Tuple[BehaviourTree, ConversationState]:
118119

119-
# Load config
120120
config = load_config(args.config_path)
121121

122122
llm: Optional[OpenAI] = None

0 commit comments

Comments
 (0)