Skip to content

Bug: "Instant Menu Speed" option in alttp is mistakenly a list and not an int #6229

@RubixDev

Description

@RubixDev

What happened?

While developing my own options creator, i've noticed that the option_instant of A Link To The Past's "Menu Speed" setting has a sneaky comma behind it, causing it to be parsed as a list rather than an int. See here:

option_instant = 1,

I don't know if this affects the official archipelago tools, given how lenient python often is, but i still believe this should be fixed.

What were the expected results?

I expected all possible values of Choice option types to always be of type int, which also seems to be the case everywhere else.

Software

Local generation

Metadata

Metadata

Assignees

No one assigned

    Labels

    is: bug/fixIssues that are reporting bugs or pull requests that are fixing bugs.

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions