google / google/python-fire

Type information should be visible in help screen

Ouverte
#239 1 commentaire 4 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Python
Étoiles
28.2k
Forks
1.5k
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Function signatures containing typing information should also show this in the help screen. For instance, I would expect the `--help` screen for `def main(arg1: str, arg2=False):` to indicate arg1 is a string type and arg2 is a bool type with default false.

I think the easiest way to make this happen is to:
- _Always_ use type information when available (even if it contradicts the default argument, since this is a developer issue -- not worth hashing out in `fire`).

However, a common programming paradigm is to write something like the following:
```python
def main(arg1: List[str] = None):
if arg1 is None:
arg1 = ['hi', 'there']
```
You can probably follow Pycharm's flow, where they redefine arg1 as Optional[List[str]]. I think this is the only valid case where the provided default doesn't match the type hint.

- Use the type of the default argument if no type information is given
- Provide no type information if both the above are missing.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.