fix: error when vault's rel_dir is a parent directory
Issue raised here, with a very simple fix actually—just normalise the path before processing.
We could look at having an abs_path argument too, but I don't want to change the API or anything right now. This is an easy change, that simply allows the parent directory (..) to be part of the rel_dir.
@jcformanek maybe you could take a quick look at this, to approve? Simple test is to have your vault in the parent directory (..)—by using the old flashbax version, an error will be thrown, but this change should fix 💯
Why are we type ignoring?
@EdanToledo apologies, solved this elsewhere with https://github.com/instadeepai/flashbax/pull/35. Need to update here :)
@EdanToledo fixed. Just waiting to confirm that mypy is now happy 🚀
@EdanToledo when you're ready: 🤾🏻♂️