feat: add new "with_path" module, which accepts and returns paths instead #18
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
add a module named "with_path"
add function "with_path::tilde_with_context" that uses Paths instead
"with_path::tilde" that uses Paths instead
fixes #15
i tried my luck with fixing #15, but because this is a massive single-file library, i opted to using a sub-module to keep it clean, also the functions in the sub-module currently have the same name, because they are literally the same only that they accept & return a
Path
insteadalso,
with_path::tilde_with_context
was changed so that it works without problems onPath
the tests have also been copied and extended for the new functions
in the future i would recommend to maybe switch to the
with_path
implementation completely so that there is less confusion and becausestr
/String
can be used in aAsRef<Path>