std::filesystem::path is used when loading assemblies, so it makes more sense to use this for the filewatcher too, instead of converting a std::filesystem::path to std::string. This is natively supported by the library.
Proposed fix _(Make sure you've read [on how to contribute]
Changed the occurences of std::string to std::filesystem::path where the filewatcher is concerned. Although std::string is used in other places, this didn't involve the filewatcher and have been left as is.
Additional context
Tested if it still works, which it should since this is not really a breaking change and is supported by the library.
Worked fine when changing a file in the sandbox project and recompiling while running.
Describe the issue (if no issue has been made)
std::filesystem::path is used when loading assemblies, so it makes more sense to use this for the filewatcher too, instead of converting a std::filesystem::path to std::string. This is natively supported by the library.
Also see: https://github.com/ThomasMonkman/filewatch#4
PR impact (Make sure to add closing keywords)
List of related issues/PRs this will solve:
Proposed fix _(Make sure you've read [on how to contribute]
Changed the occurences of std::string to std::filesystem::path where the filewatcher is concerned. Although std::string is used in other places, this didn't involve the filewatcher and have been left as is.
Additional context
Tested if it still works, which it should since this is not really a breaking change and is supported by the library. Worked fine when changing a file in the sandbox project and recompiling while running.