Commit 26b9f6fa authored by Tristan Van Berkom's avatar Tristan Van Berkom

_stream.py: Clarifying ambiguous FIXME comment about required artifacts

parent d835c37f
Pipeline #26001259 passed with stages
in 30 minutes and 56 seconds
...@@ -879,9 +879,12 @@ class Stream(): ...@@ -879,9 +879,12 @@ class Stream():
# Set the "required" artifacts that should not be removed # Set the "required" artifacts that should not be removed
# while this pipeline is active # while this pipeline is active
# #
# FIXME: The set of required artifacts should probably be # FIXME: The set of required artifacts is only really needed
# what's in `selected`, but this does not seem to work # for build and pull tasks.
# for some reason #
# It must include all the artifacts which are required by the
# final product. Note that this is a superset of the build plan.
#
self._artifacts.append_required_artifacts((e for e in self._pipeline.dependencies(elements, Scope.ALL))) self._artifacts.append_required_artifacts((e for e in self._pipeline.dependencies(elements, Scope.ALL)))
if selection == PipelineSelection.PLAN and dynamic_plan: if selection == PipelineSelection.PLAN and dynamic_plan:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment