Add minor improvements in readability in CI config

lib/gitlab/ci/config/node/configurable.rb

@@ -26,9 +26,7 @@ module Gitlab
           private
 
           def create_node(key, factory)
-            factory.with(value: @config[key])
+            factory.with(value: @config[key], key: key, parent: self)
-            factory.with(parent: self)
-            factory.with(key: key)
 
             factory.create!
           end

lib/gitlab/ci/config/node/stages.rb

@@ -13,7 +13,7 @@ module Gitlab
           end
 
           def self.default
-            %w(build test deploy)
+            %w[build test deploy]
           end
         end
       end

lib/gitlab/ci/config/node/validator.rb

@@ -23,6 +23,7 @@ module Gitlab
 
           def unknown_keys
             return [] unless config.is_a?(Hash)
+
             config.keys - @node.class.nodes.keys
           end
 

spec/lib/ci/gitlab_ci_yaml_processor_spec.rb

@@ -551,8 +551,8 @@ module Ci
               config_processor = GitlabCiYamlProcessor.new(config, path)
 
               ##
-              # When variables config is empty, we asumme this is a correct,
+              # When variables config is empty, we assume this is a valid
-              # see issue #18775
+              # configuration, see issue #18775
               #
               expect(config_processor.job_variables(:rspec))
                 .to be_an_instance_of(Array).and be_empty

spec/lib/gitlab/ci/config/node/boolean_spec.rb

@@ -21,7 +21,7 @@ describe Gitlab::Ci::Config::Node::Boolean do
     end
 
     context 'when entry value is not valid' do
-      let(:config) { [ 'incorrect' ] }
+      let(:config) { ['incorrect'] }
 
       describe '#errors' do
         it 'saves errors' do

spec/lib/gitlab/ci/config/node/configurable_spec.rb

@@ -22,7 +22,6 @@ describe Gitlab::Ci::Config::Node::Configurable do
       validator.validate
     end
 
-
     context 'when node validator is invalid' do
       let(:instance) { node.new('ls') }