google/python-fire

Only plain literals should be parsed

Open

#97 aperta il 12 ott 2017

Vedi su GitHub
 (9 commenti) (0 reazioni) (0 assegnatari)Python (1418 fork)batch import
bughelp wanted

Metriche repository

Star
 (26.214 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

@dbieber correctly pointed out in #95 that there is a more general problem with the parsing code. This is a follow up issue to that issue which only dealt with the most common case of a single argument.

Here are some cases demonstrating

$ python3 -c "import fire.parser as p; v = p.DefaultParseValue('[1+1]'); print(type(v), v)"
<class 'list'> [2]
$ python3 -c "import fire.parser as p; v = p.DefaultParseValue('{key: 1+1}'); print(type(v), v)"
<class 'dict'> {'key': 2}

Some options are:

  1. Accept the existing behaviour.
  2. Always parse the whole argument as a string by doing raise ValueError in ``fire.parser._LiteralEval`
  3. Parse only the sub expressions as strings

For 2. and 3. I think these would be the valid types from the ast module.

  • Str
  • Name (converted to Str)
  • Num
  • List
  • Tuple
  • Dict
  • Set
  • UnaryOp

Guida contributor

Direzione di ricerca
Il problema riguarda la funzione DefaultParseValue in fire/parser.py. La correzione comporta la modifica della funzione LiteralEval per accettare solo specifici nodi AST (Str, Name, Num, List, Tuple, Dict, Set, UnaryOp). Rivedere la discussione in #95 e l'implementazione attuale per decidere quale opzione implementare. La correzione scelta dovrebbe sollevare ValueError per espressioni non supportate o analizzare le sottoespressioni come stringhe.
Tech stack
python
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70