Skip to content

Commit 65d363a

Browse files
Fix ruff issues after merge
1 parent 5805db0 commit 65d363a

5 files changed

Lines changed: 119 additions & 128 deletions

File tree

src/conductor/shared/worker/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
__all__ = [
1212
"NonRetryableException",
1313
"TaskOptions",
14-
"task_options",
15-
"get_task_options",
1614
"apply_task_options_to_task_def",
15+
"get_task_options",
16+
"task_options",
1717
]

src/conductor/shared/worker/task_definition_helper.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,7 @@ def apply_task_options_to_task_def(task_def, task_options: TaskOptions) -> None:
3535
task_def.rate_limit_per_frequency = task_options.rate_limit_per_frequency
3636

3737
if task_options.rate_limit_frequency_in_seconds is not None:
38-
task_def.rate_limit_frequency_in_seconds = (
39-
task_options.rate_limit_frequency_in_seconds
40-
)
38+
task_def.rate_limit_frequency_in_seconds = task_options.rate_limit_frequency_in_seconds
4139

4240
if task_options.concurrent_exec_limit is not None:
4341
task_def.concurrent_exec_limit = task_options.concurrent_exec_limit

src/conductor/shared/worker/task_options.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -98,10 +98,7 @@ def _validate_parameters(
9898
if rate_limit_per_frequency is not None and rate_limit_per_frequency < 0:
9999
raise ValueError("rate_limit_per_frequency must be >= 0")
100100

101-
if (
102-
rate_limit_frequency_in_seconds is not None
103-
and rate_limit_frequency_in_seconds < 0
104-
):
101+
if rate_limit_frequency_in_seconds is not None and rate_limit_frequency_in_seconds < 0:
105102
raise ValueError("rate_limit_frequency_in_seconds must be >= 0")
106103

107104
if concurrent_exec_limit is not None and concurrent_exec_limit < 0:
@@ -133,9 +130,7 @@ def to_dict(self):
133130
if self.rate_limit_per_frequency is not None:
134131
result["rate_limit_per_frequency"] = self.rate_limit_per_frequency
135132
if self.rate_limit_frequency_in_seconds is not None:
136-
result["rate_limit_frequency_in_seconds"] = (
137-
self.rate_limit_frequency_in_seconds
138-
)
133+
result["rate_limit_frequency_in_seconds"] = self.rate_limit_frequency_in_seconds
139134
if self.concurrent_exec_limit is not None:
140135
result["concurrent_exec_limit"] = self.concurrent_exec_limit
141136
if self.timeout_policy is not None:

0 commit comments

Comments
 (0)