The order of arguments matters in buildInputs, sadly. In this case, pkgs.ormolu was shadowed by the Ormolu brought by hpkgs.haskell-language-server. The versions are quite different: nixpkgs contains Ormolu 0.3 while HLS brings Ormolu 0.5. Since most things rely on nixpkgs's Ormolu, I think it makes more sense to ensure that it is the one picked up by the pre-commit hooks and available in the environment. The current PR fixes that issue by reordering the build inputs, and leaves a note for our future selves.
The order of arguments matters in
buildInputs, sadly. In this case,pkgs.ormoluwas shadowed by the Ormolu brought byhpkgs.haskell-language-server. The versions are quite different: nixpkgs contains Ormolu 0.3 while HLS brings Ormolu 0.5. Since most things rely on nixpkgs's Ormolu, I think it makes more sense to ensure that it is the one picked up by the pre-commit hooks and available in the environment. The current PR fixes that issue by reordering the build inputs, and leaves a note for our future selves.