Skip to content

Commit 5cbe1aa

Browse files
committed
Fixup rebase
1 parent 918d5ce commit 5cbe1aa

File tree

4 files changed

+8
-4
lines changed

4 files changed

+8
-4
lines changed

metaflow/cli.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ def start(
475475
raise ctx.obj.delayed_config_exception
476476

477477
# Init all values in the flow mutators and then process them
478-
for decorator in ctx.obj.flow._flow_state[FlowStateItems.FLOW_MUTATORS]:
478+
for decorator in ctx.obj.flow._flow_mutators:
479479
decorator.external_init()
480480

481481
new_cls = ctx.obj.flow._process_config_decorators(config_options)
@@ -596,7 +596,7 @@ def start(
596596
ctx.obj.echo,
597597
ctx.obj.flow_datastore,
598598
{
599-
k: v if plain_flag else ConfigValue(v)
599+
k: v if plain_flag or v is None else ConfigValue(v)
600600
for k, (v, plain_flag) in ctx.obj.flow.__class__._flow_state[
601601
FlowStateItems.CONFIGS
602602
].items()

metaflow/decorators.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -803,7 +803,7 @@ def _init_step_decorators(
803803
# and then the step level ones to maintain a consistent order with how
804804
# other decorators are run.
805805

806-
for deco in cls._flow_state[FlowStateItems.FLOW_MUTATORS]:
806+
for deco in cls._flow_mutators:
807807
if isinstance(deco, FlowMutator):
808808
inserted_by_value = [deco.decorator_name] + (deco.inserted_by or [])
809809
mutable_flow = MutableFlow(

metaflow/flowspec.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -331,6 +331,10 @@ def _flow_decorators(self):
331331
# Backward compatible method to access flow decorators
332332
return self._flow_state[FlowStateItems.FLOW_DECORATORS]
333333

334+
@property
335+
def _flow_mutators(self):
336+
return self._flow_state[FlowStateItems.FLOW_MUTATORS]
337+
334338
@classmethod
335339
def _check_parameters(cls, config_parameters=False):
336340
seen = set()

metaflow/runner/click_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,7 @@ def _compute_flow_parameters(self):
532532
# We ignore any errors if we don't check the configs in the click API.
533533

534534
# Init all values in the flow mutators and then process them
535-
for decorator in self._flow_cls._flow_state[FlowStateItems.FLOW_MUTATORS]:
535+
for decorator in self._flow_cls._flow_mutators:
536536
decorator.external_init()
537537

538538
new_cls = self._flow_cls._process_config_decorators(

0 commit comments

Comments
 (0)