Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1036)

Unified Diff: recipe_engine/step_runner.py

Issue 2934583003: [step] Remove config indirection. (Closed)
Patch Set: fix test Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: recipe_engine/step_runner.py
diff --git a/recipe_engine/step_runner.py b/recipe_engine/step_runner.py
index 421072e31e9884a3867486abd3e55116fb6b6e4e..d14d636d1ed3270c891a4d3c788f1b9f7e8f8644 100644
--- a/recipe_engine/step_runner.py
+++ b/recipe_engine/step_runner.py
@@ -92,7 +92,7 @@ class StepRunner(object):
"""Constructs an OpenStep object which can be used to actually run a step.
Args:
- step_config (StepConfig): The step data.
+ step_config (recipe_api.StepClient.StepConfig): The step data.
Returns: an OpenStep object.
"""
@@ -549,7 +549,7 @@ Placeholders = collections.namedtuple('Placeholders',
# Result of 'render_step'.
#
# Fields:
-# config (recipe_api.StepConfig): The step configuration.
+# config (recipe_api.StepClient.StepConfig): The step configuration.
# placeholders (Placeholders): Placeholders for this rendered step.
# followup_annotations (list): A list of followup annotation, populated during
# simulation test.
@@ -565,7 +565,7 @@ def render_step(step_config, step_test):
"""Renders a step so that it can be fed to annotator.py.
Args:
- step_config (StepConfig): The step config to render.
+ step_config (recipe_api.StepClient.StepConfig): The step config to render.
step_test: The test data json dictionary for this step, if any.
Passed through unaltered to each placeholder.
« no previous file with comments | « recipe_engine/run.py ('k') | recipe_engine/stream.py » ('j') | recipe_modules/step/api.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698