Project

General

Profile

Refactor #21360

Remove legacy code for Ruby <= 2.0 from hammer-cli-foreman

Added by Tomáš Strachota about 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Hammer core
Target version:
-
Difficulty:
easy
Triaged:
No
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Hammer plugin for the Foreman no longer supports ruby 2.0, there's no point of keeping the historical code.

From my search there's following places containing some compatibility code that should be removed:

./hammer/hammer-cli-foreman/lib/hammer_cli_foreman/option_builders.rb:
   93  
   94      def unique(resources)
   95:       # ruby 1.8 hack - it does not support passing blocks to Array#uniq
   96        resources.inject({}) do |h, r|
   97          h.update(r.name => r)

./hammer/hammer-cli-foreman/Gemfile:
    4  
    5  gem 'gettext', '>= 3.1.3', '< 4.0.0'
    6: if RUBY_VERSION < '2.1.0'
    7    gem 'fast_gettext', '< 1.2.0'
    8  end


Related issues

Related to Hammer CLI - Refactor #21359: Remove legacy code for Ruby < 2.0 from hammer-cliClosed

Associated revisions

Revision 0db1790e (diff)
Added by Oleh Fedorenko over 1 year ago

Fixes #21360 - Remove legacy code for Ruby < 2.0

History

#1 Updated by Tomáš Strachota about 2 years ago

  • Related to Refactor #21359: Remove legacy code for Ruby < 2.0 from hammer-cli added

#2 Updated by Oleh Fedorenko over 1 year ago

  • Triaged set to No
  • Assignee set to Oleh Fedorenko
  • Status changed from New to Assigned

#3 Updated by The Foreman Bot over 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/372 added

#4 Updated by Tomáš Strachota over 1 year ago

  • Subject changed from Remove legacy code for Ruby < 2.0 from hammer-cli-foreman to Remove legacy code for Ruby <= 2.0 from hammer-cli-foreman

Updating the title. The comparison sign didn't correspond with the issue description.

#5 Updated by Oleh Fedorenko over 1 year ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Tomer Brisker over 1 year ago

  • Fixed in Releases hammer-cli-foreman-0.14.0 added

Also available in: Atom PDF