➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2020-01-12 12:58 by BTaskaya, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Messages (2) | |||
|---|---|---|---|
| msg359847 - (view) | Author: Batuhan Taskaya (BTaskaya) *  |
Date: 2020-01-12 12:58 | |
>>> tuple(difflib.context_diff(["abc"], ["bcd"], fromfile=Path("example.py")))
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/local/lib/python3.9/difflib.py", line 1254, in context_diff
_check_types(a, b, fromfile, tofile, fromfiledate, tofiledate, lineterm)
File "/usr/local/lib/python3.9/difflib.py", line 1301, in _check_types
raise TypeError('all arguments must be str, not: %r' % (arg,))
TypeError: all arguments must be str, not: PosixPath('example.py')
IMHO to and from file arguments should accept PathLike objects. If agreed I can prepare a patch.
|
|||
| msg359885 - (view) | Author: Raymond Hettinger (rhettinger) * |
Date: 2020-01-13 03:43 | |
Elsewhere users of Path objects must explicitly coerce to str. For example: json.dumps(str(Path("example.py"))). This makes good sense to me. We don't want to add an dependency on pathlib just to recognize Path objects.
|
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:25 | admin | set | github: 83492 |
| 2020-01-13 03:43:34 | rhettinger | set | status: open -> closed nosy: + rhettinger messages: + msg359885 resolution: rejected stage: resolved |
| 2020-01-12 13:49:29 | xtreak | set | nosy:
+ tim.peters |
| 2020-01-12 12:58:02 | BTaskaya | create | |