Project

General

Profile

Bug #35300

Tracker #32684: Ruby 3.0 support

Incorrect method signatures with keyword arguments

Added by Ewoud Kohl van Wijngaarden 4 months ago. Updated 1 day ago.

Status:
Closed
Priority:
Normal
Category:
Rails
Target version:
-

Description

In Ruby 3.0 keyword arguments are handled separate from positional arguments. https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/ has more about it.


Related issues

Related to Ansible - Bug #35579: test/functional/api/v2/ansible_inventories_controller_test.rb doesn't use keyword argumentsReady For Testing
Has duplicate Foreman - Bug #32688: app/models/concerns/has_many_common.rb is incompatible with Ruby 3Duplicate

Associated revisions

Revision 9089087e (diff)
Added by Ewoud Kohl van Wijngaarden 3 months ago

Fixes #35300 - Pass keyword arguments correctly (#9332)

In Ruby 3.0 keyword arguments are handled separate from positional
arguments1. This uses kwargs and passes it along.

[1]: https://www.ruby-lang.org/en/news/2019/12/12/separation-of-positional-and-keyword-arguments-in-ruby-3-0/

Revision 6eaf16e1 (diff)
Added by Ewoud Kohl van Wijngaarden 2 months ago

Refs #35300 - Pass options as keyword arguments (#9441)

In 9089087ed7b2a7da33b73711aa5f457fe06adfc2 various keyword arguments
were fixed, but some remained.

Fixes: 9089087ed7b2a7da33b73711aa5f457fe06adfc2

Revision d7ce6881 (diff)
Added by Ewoud Kohl van Wijngaarden 28 days ago

Refs #35300 - Pass arguments as keyword arguments

History

#1 Updated by The Foreman Bot 4 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9332 added

#2 Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9382 added

#3 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Has duplicate Bug #32688: app/models/concerns/has_many_common.rb is incompatible with Ruby 3 added

#4 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Parent task set to #32684
  • Category set to Rails

#5 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases 3.5.0 added

#6 Updated by Ewoud Kohl van Wijngaarden 3 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9441 added

#8 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Related to Bug #35579: test/functional/api/v2/ansible_inventories_controller_test.rb doesn't use keyword arguments added

#9 Updated by The Foreman Bot about 1 month ago

  • Pull request https://github.com/theforeman/foreman/pull/9472 added

#10 Updated by Ewoud Kohl van Wijngaarden 1 day ago

  • Triaged changed from No to Yes

Also available in: Atom PDF