Skip to content

Remove unused default argument from ruby class

Jacob Vosmaer requested to merge jv-remove-default-argument into master

I noticed this in !1748 (merged).

We added a new ruby class that takes 1 argument in its initializer, and that argument has a default value. This means you are allowed to write either PushOptions.new(foo) or PushOptions.new. However, the second form is never used. That is confusing.

This MR removes the support for the unused second form (PushOptions.new).

Merge request reports