castwide/gamefic

Keyword Arguments for Room#connect

Closed this issue · 1 comments

The Room#connect method accepts up to four positional arguments. Keyword arguments would be easier to read and maintain. The declaration could look something like this:

def connect(destination:, direction: nil, type: Portal, two_way: true)

This is a breaking change, so we need to decide how to provide an upgrade path and whether it should be considered a major or minor revision.

The latest version supports keyword arguments and is backwards-compatible with the original version.