Spot the bug: ``` process = Process( ar...
# random
b
Spot the bug:
Copy code
process = Process(
        argv=("/bin/ln", "-s", "dest", "source"),
        output_files="source",
        description="",
        working_directory="",
    )
w
Isn't output_files an array?
Or i guess.... a char string
b
yeah it's an iterable of strings
Which a string is an iterable of strings 🤦‍♂️
c
["s", "o", "u", "r", "c", "e"]
lovely 😛
w
pants v1 used to have a single special-case for this, to assert that the iterable wasn’t a string
if there is a base
IterableField
, could add some validation there: “iterable but !str”
👍 1
a
lol. I've made this mistake myself. took me hours to work out what was going on
b
That would be a nice safety check