Closed jamesmyatt closed 2 years ago
Thanks a lot for your report!
I don't think it's a good idea to parse file names using regular expressions. You're better off with pathlib, in my experience.
You're absolutely right. I've been migrating to use pathlib.Path recently and this is the one I have to fix in the same way.
I've just committed several changes (5b9bae8 and related ones) may resolve this issue together with adding windows test targets for running CI (5456fbc). Could you please test the latest one in the next branch if it' possible, or wait for the next release which I'm planning to release within a few days?
Thanks. I'll take a look when it's released. I'm not sure when expand_from_path
is used but I think it can be simplified. I'll try to take another look some time.
I was also having this issue. I've installed from the branch next
and the issue is resolved! @ssato
Seems fixed. Thanks
When I do
import anyconfig
using v0.11.1 and Python 3.9.7 on Windows, I get the following. I think this PR fixes it.Although, TBH, I don't think it's a good idea to parse file names using regular expressions. You're better off with pathlib, in my experience. Regardless, there's an issue with
split_path_by_marker
that, ifsplit_re
needs to be updated based onmarker
andpath_sep
, i.e. they're not independent inputs.