Be more flexible on accepted paths

[felixfontein]

No description provided.

[gotmax23]

Thanks for cleaning this up! I think we should standardize on StrPath so we can use os.path.join ¹ and others and not have to worry about converting between str and bytes. It's confusing if some but not all functions support bytes paths.

Footnotes

1. If one argument to os.path.join is bytes, all the others have to be. ↩

[felixfontein]

Thanks for cleaning this up! I think we should standardize on StrPath so we can use os.path.join 1 and others and not have to worry about converting between str and bytes. It's confusing if some but not all functions support bytes paths.

We already use StrOrBytesPath in multiple places (src/antsibull_core/yaml.py, src/antsibull_core/subprocess_util.py, src/antsibull_core/utils/io.py). At least for hashing I would also use it, since it's very similar to the others. For dependency_files I don't mind switching to StrPath (I'll push a commit for that soon).

[felixfontein]

@gotmax23 thanks for reviewing this!