tshirtman, @yantor3d I would argue for fully spelled out argument names, i.e source, target (or destination?), unless it's a convention in your codebase to use shortened names.
I would have used pathlib.Path instead of os.path, but that's an implementation detail, shouldn't matter (much) to users, thought returning a Path object is generally better than a string representing a path.
Could the function name be clearer regarding the usage?
Add comment