You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[Enumerator::_Each] is not a replacement for _Each
⸺ #1915 (comment)
Namespaced or not, it might be more organizational to dissolve builtin.rbs’s convenience types to more relevant places, such as enumerable.rbs in ::_Each and _ToInt&int in integer.rbs.
I think _MethodName should be restricted to interfaces that implement that method_name minimally.
⸺ #424 (comment)
Would it be worthy to add lower-case interfaces _example?
And start a new convention where _method_name implements #method_name and _ClassName is a duck type.
Examples:
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Namespaced or not, it might be more organizational to dissolve
builtin.rbs
’s convenience types to more relevant places, such asenumerable.rbs
in::_Each
and_ToInt
&int
ininteger.rbs
.Would it be worthy to add lower-case interfaces
_example
?And start a new convention where
_method_name
implements#method_name
and_ClassName
is a duck type.Examples:
rbs/core/builtin.rbs
Lines 206 to 220 in 45b34bd
Beta Was this translation helpful? Give feedback.
All reactions