Project

General

Profile

Refactor #21360

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

Added by Tomáš Strachota 11 months ago. Updated 22 days 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-cliReady For Testing

Associated revisions

Revision 0db1790e (diff)
Added by Oleh Fedorenko about 2 months ago

Fixes #21360 - Remove legacy code for Ruby < 2.0

History

#1 Updated by Tomáš Strachota 11 months ago

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

#2 Updated by Oleh Fedorenko 3 months ago

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

#3 Updated by The Foreman Bot 3 months 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 about 2 months 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 about 2 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Tomer Brisker 22 days ago

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

Also available in: Atom PDF