search

transparencia-mg / dpckan

MIT License
5 stars 4 forks source link

Incluir flag para possibilitar `sys.exit(1)` quando comandos dpckan retornarem situações passíveis de parar execução como erro #178

Closed gabrielbdornas closed 2 years ago

gabrielbdornas commented 2 years ago

Exemplo dpckan update caso"nothing to be updated..."

gabrielbdornas commented 2 years ago

Funcionando no dataset acordo judicial da vale.

Run dpckan --datastore --exit-code --ckan-host "$CKAN_HOST" --ckan-key "$CKAN_KEY" dataset update
  dpckan --datastore --exit-code --ckan-host "$CKAN_HOST" --ckan-key "$CKAN_KEY" dataset update
  shell: sh -e {0}
  env:
    CKAN_HOST: ***
    CKAN_KEY: ***
Nothing to be updated in dataset ***/dataset/acordo-judicial-reparacao-vale.
Error: Process completed with exit code 1.
fjuniorr commented 2 years ago

Não vi a implementação, mas o nome desse argumento não faz sentido pra mim. Todo programa CLI deve retornar um exit_code quando finaliza a execução, seja de sucesso ou fracasso.

Se essa flag está controlando situações específicas em que uma falha deve ser retornada (exit code > 0), então ela deve indicar em quais situações isso está acontecendo.