Skip to content

Conversation

@mrboring
Copy link
Contributor

@mrboring mrboring commented Aug 7, 2025

I think you missed this:

    /// Implicitly convert an ArgumentArity instance to an Arity DU.
    static member inline op_Implicit(argumentArity: ArgumentArity) : Arity = 
        match argumentArity.MinimumNumberOfValues, argumentArity.MaximumNumberOfValues with
        | 0, 0 -> Zero
        | 0, 1 -> ZeroOrOne               // <<<<<<<<<<<
        | 0, max when max = 100_000 -> ZeroOrMore
        | min, max when min = max -> ExactlyOne
        | min, max when min = 1 && max = 100_000 -> OneOrMore
        | min, max when min > 0 && max = 100_000 -> MinimumNumberOfValues min
        | min, max when min = 0 && max < 100_000 -> MaximumNumberOfValues max
        | _ -> ArgumentArity (argumentArity.MinimumNumberOfValues, argumentArity.MaximumNumberOfValues)

@JordanMarr JordanMarr merged commit 1d145b4 into JordanMarr:main Aug 16, 2025
1 check passed
@JordanMarr
Copy link
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants