Skip to content

Commit 54bd3b4

Browse files
committed
Don't run Fourmolu in debug mode
It prints to stderr, due to uses of `traceM`, and it's not nice to work around this downstream. It's of questionable utility anyway. The fact that it mostly prints information about extensions, and does so in a hard-to-read format (a string displayed as a list!) indicates that it probably isn't widely used.
1 parent b1e23e7 commit 54bd3b4

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

plugins/hls-fourmolu-plugin/src/Ide/Plugin/Fourmolu.hs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,13 +82,12 @@ provider recorder plId ideState typ contents fp fo = withIndefiniteProgress titl
8282
(exitCode, out, err) <- -- run Fourmolu
8383
readCreateProcessWithExitCode
8484
( proc "fourmolu" $
85-
["-d"]
85+
map ("-o" <>) fileOpts
8686
<> mwhen noCabal ["--no-cabal"]
8787
<> catMaybes
8888
[ ("--start-line=" <>) . show <$> regionStartLine region
8989
, ("--end-line=" <>) . show <$> regionEndLine region
9090
]
91-
<> map ("-o" <>) fileOpts
9291
){cwd = Just $ takeDirectory fp'}
9392
contents
9493
case exitCode of
@@ -114,7 +113,7 @@ provider recorder plId ideState typ contents fp fo = withIndefiniteProgress titl
114113
defaultConfig
115114
{ cfgDynOptions = map DynOption fileOpts
116115
, cfgRegion = region
117-
, cfgDebug = True
116+
, cfgDebug = False
118117
, cfgPrinterOpts =
119118
fillMissingPrinterOpts
120119
(printerOpts <> lspPrinterOpts)

0 commit comments

Comments
 (0)