chagnes from review

yast · Oct 12, 2016 · 2814e51 · 2814e51
1 parent 69fb68e
commit 2814e51
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -5,7 +5,7 @@ before_install:
     # disable rvm, use system Ruby
     - rvm reset
     - wget https://raw.githubusercontent.com/yast/yast-devtools/master/travis-tools/travis_setup.sh
-    - sh ./travis_setup.sh -p "rake yast2-devtools yast2-testsuite yast2 yast2-storage yast2-proxy yast2-country yast2-packager" -g "rspec:3.3.0 yast-rake gettext rubocop:0.41.2 simplecov:0.10.0 coveralls"
+    - sh ./travis_setup.sh -p "rake yast2-devtools yast2-testsuite yast2 yast2-storage yast2-proxy yast2-country yast2-packager" -g "rspec:3.3.0 yast-rake gettext rubocop:0.41.2 simplecov:0.10.0 coveralls cfa"
 script:
     - rubocop
     - rake check:syntax

diff --git a/src/modules/Host.rb b/src/modules/Host.rb
@@ -194,8 +194,6 @@ def Export
 
       # Filter out IPs with empty hostname (so that valid autoyast
       # profile is created)(#335120)
-      # FIXME: this also removes records with empty names from @hosts. Such
-      # side effect is unexpected and should be removed.
       exported_hosts.keep_if { |_, names| !names.empty? }
 
       { "hosts" => exported_hosts }