google / google/python-fire

Strings args do not need to be parsed.

Open
#459 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
28.2k
Forks
1.5k
PR merge metrics
No merged PRs in 30d

Description

```
def test(a, b):
print(a, b)

if __name__ == "__main__":
fire.Fire(test)
```

`python .\test.py 1 '2 3'`
```1 2 3```

`python .\test2.py 1 '""2 3""'`
`1 2 3`

`python .\test.py 1 '--2 3'`
```ERROR: The function received no value for the required argument: b```

`python .\test.py 1 '\"2 3\"'`
```
1 "2
ERROR: Could not consume arg: 3"
```

`python .\test.py 1 '\"\"2 3\"\"'`
`1 ""2 3""`

The correct behavior should be equivalent to `print(1, "2 3")`, so strings args do not need to be parsed.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.