Skip to content

CLI: passe les arguments à `insitu import` et `insitu harvest` sans `--paths` ou `--names` devant

Ronan Amicel requested to merge fix-cli-params into main

Contexte

  • La commande insitu import ne prend plus les chemins directement en argument, mais nous demande de préfixer par --paths alors que paths est bien un Argument et pas une Option.
$ insitu import --help

 Usage: insitu import [OPTIONS]

 Import local file(s).

╭─ Options ─────────────────────────────────────────────────────────────────────────────────────────╮
│ --paths                                PATH  [default: None]                                      │
│ --dataset                              TEXT  [default: None]                                      │
│ --sheetname                            TEXT  [default: None]                                      │
│ --last-modified                        TEXT  [default: None]                                      │
│ --params                               TEXT  [default: None]                                      │
│ --archive          --no-archive              [default: no-archive]                                │
│ --export           --no-export               [default: no-export]                                 │
│ --dry-run          --no-dry-run              [default: no-dry-run]                                │
│ --interactive      --no-interactive          [default: no-interactive]                            │
│ --help                                       Show this message and exit.                          │
╰───────────────────────────────────────────────────────────────────────────────────────────────────╯

Contenu

On parle différemment à typer pour qu’il accepte de revenir au comportement souhaité.

$ insitu import --help

 Usage: insitu import [OPTIONS] [PATHS]...

 Import local file(s).

╭─ Arguments ─────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│   paths      [PATHS]...  path(s) of files or directories to import [default: None]                                  │
╰─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
╭─ Options ───────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ --dataset                              TEXT  [default: None]                                                        │
│ --sheetname                            TEXT  [default: None]                                                        │
│ --last-modified                        TEXT  [default: None]                                                        │
│ --params                               TEXT  [default: None]                                                        │
│ --archive          --no-archive              [default: no-archive]                                                  │
│ --export           --no-export               [default: no-export]                                                   │
│ --dry-run          --no-dry-run              [default: no-dry-run]                                                  │
│ --interactive      --no-interactive          [default: no-interactive]                                              │
│ --help                                       Show this message and exit.                                            │
╰─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
Edited by Ronan Amicel

Merge request reports