Move and improvement comment in pipeline fixturesfix/pipeline-seeds-in-dev-env

1 files changed, 4 insertions, 5 deletions

diff --git a/db/fixtures/development/14_pipelines.rb b/db/fixtures/development/14_pipelines.rb
index 36b0d16bfb9..49e6e2361b1 100644
--- a/db/fixtures/development/14_pipelines.rb
+++ b/db/fixtures/development/14_pipelines.rb
@@ -74,6 +74,10 @@ class Gitlab::Seeder::Pipelines
       .merge(commands: '$ build command')
 
     Ci::Build.create!(attributes).tap do |build|
+      # We need to set build trace and artifacts after saving a build
+      # (id required), that is why we need `#tap` method instead of passing
+      # block directly to `Ci::Build#create!`.
+
       setup_artifacts(build)
       setup_build_log(build)
       build.save
@@ -93,11 +97,6 @@ class Gitlab::Seeder::Pipelines
   end
 
   def setup_build_log(build)
-    ##
-    # We need to set build trace after saving a build (id required)
-    # That is why we need `#tap` method instead of passing block
-    # directly to `Ci::Build#create!`.
-    #
     if %w(running success failed).include?(build.status)
       build.trace = FFaker::Lorem.paragraphs(6).join("\n\n")
     end