Closed lauraschauer closed 1 year ago
Hey @lauraschauer , thank you for your PR!
The change looks good from my POV. The only thing I would like to ask you is to run the pre-commit hooks before committing (the easiest way is to run nix-shell
to install them). This runs in particular ormolu for the formatting which was missed here as you can see from this failed CI run (it is a super small thing, only about the order of the imports).
Once that is done and CI passes I am happy to merge :smile:
Okay, this should hopefully be fixed now :)
Change import statement from
import Path.Posix
toimport Path
to avoid bugs when building the library on a Windows OS. ImportingPath
on its own is usually better: Hackage Path.