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

set-var(): Check input, die on errors #882

Merged
merged 1 commit into from
Feb 11, 2023
Merged

set-var(): Check input, die on errors #882

merged 1 commit into from
Feb 11, 2023

Commits on Feb 9, 2023

  1. set-var(): Check input, die on errors 

    Previously, set_var() had no input checking, combined with
the point that, via vars, set_var() is user facing, this
can lead to easy user errors.

Now, input is checked:
- Parameter 1 is required.
- Parameter 1 cannot contain a space.
- Less-than 3 input parameters are expected.
- Quote the expansion of the first occurence of parameter 1
  in the evaluated expression.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
    @TinCanTech
    TinCanTech committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    66d282e View commit details
    Browse the repository at this point in the history