Skip to content

[v2] remove unnecessary json lists from param values #136

Description

@mimir-d

Currently, all of the parameters are expected to be lists, even when it is unnecessary (like the "binary" below).

{
    "name": "gathercmd",
    "label": "make_logdirs",
    "parameters": {
        "binary": ["mkdir"], <--
        "args": ["-p", "/tmp/control_logs"]
    }
}

There is also some api supporting this Param.GetOne. This should be removed, and the descriptors updated to have single values.
Proposal

{
    "name": "gathercmd",
    "label": "make_logdirs",
    "parameters": {
        "binary": "mkdir", <--
        "args": ["-p", "/tmp/control_logs"]
    }
}

Activity

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

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingv2_overhaulbreaking feature for v2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions