Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unbound splat arguments, fixes GH-54 #59

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Unbound splat arguments, fixes GH-54 #59

wants to merge 2 commits into from

Commits on Dec 18, 2023

  1. fix!: Unbound splat arguments 

    BREAKING CHANGE: The default maxcount value for SPLAT arguments is now
unlimited. The previous default was 1. That was unexpected for many
folks and arguably wrong, but fixing it is a major breaking change and
consumers will need to adjust, hence the major version bump. The old
behaviour can be achieved by setting the maxcount to 1 again:

    cli:splat('MY_SPLAT', 'Description', nil, 1)

Additionally a limit of 999 has been resolved so unlimited arguments can
be accepted.
    @amireh @alerque
    amireh authored and alerque committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    9afba52 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into unbounded-splats

    @alerque
    alerque committed Dec 18, 2023
    Configuration menu
    Copy the full SHA
    2da9ecd View commit details
    Browse the repository at this point in the history