nfdi4plants / ARCCommander

Tool to manage your ARCs
MIT License
11 stars 9 forks source link

ArcCommander Argument design #7

Closed HLWeil closed 2 years ago

HLWeil commented 3 years ago

This issue is meant as a place for discussing possible argument design choices

Argument Query (Asking for fields via text editor prompt)

muehlhaus commented 3 years ago

Top-level

arc init - create an arc directory structure below the current directory (as git init) arc push - push all changes to the specified "upstream" (t.b.d.) arc pull - ... arc check - validate that the current directory is an arc, and output helpful messages towards compliance

arc register - (re-)register all assays and workflows

arc help

arc run - run all workflows

Investigation

arc investigation or arc i subcmd arc i edit - open editor to edit ISA in YAML (yay!)

Assay

arc assay [c|create] - create a directory assays/, prepopulate it with defaults

arc assay [rregister]

| --update

arc assay [a|add] - create plus register

arc assay [a|move] ...

arc assay [rm|remove]

arc assay edit -- edit assay metadata in Excel or

HLWeil commented 3 years ago

Some arguments are required and some are not. If required arguments are left out by the user a prompt should be opened where the user can input the missing arguments. Ideally required arguments should be marked (e.g. by an attribute). Also it would be good

chgarth commented 3 years ago

Let me point out that in all Unix systems (including macOS), it is the convention that the EDITOR (or less often, VISUAL) environment variable is used to designate the preferred editor. This works with VSCode’s “code” command line interface as well.

On 29. Oct 2020, at 18:11, Lukas Weil notifications@github.com wrote:

Some arguments are required and some are not. If required arguments are left out by the user a prompt should be opened where the user can input the missing arguments. Ideally required arguments should be marked (e.g. by an attribute). Also it would be good

if the texteditor could be opened on a stream instead of a file (check if possible) if the texteditor of choice could be set in a config file — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nfdi4plants/arcCommander/issues/7#issuecomment-718896152, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKFCTTZGYDSFP7ZD25UIGLSNGO5JANCNFSM4TAQWKPQ.

kMutagene commented 3 years ago

via #13 and #14 we now have the following structure of the arc cli.

arc <top-level-command> <top-level-command-args>

e.g.

arc init

or

arc <object> <subcommand-verb> <subcommand-verb-args>

where <object> is one of the following:

and <subcommand-verb> models what to do with the object, e.g

arc study init

will initialize a new empty study in the ARC.

Subcommand verbs

while not all object subcommands support all verbs, here is a list of the verbs and what to expect when they are applicable for an <object>:

HLWeil commented 2 years ago

Will close this for now, as the argument structure seems to have stabilized