search

lassik / emacs-format-all-the-code

Auto-format source code in many languages with one command
https://melpa.org/#/format-all
MIT License
604 stars 105 forks source link

Why does ormolu config not have --stdin-input-file like fourmolu? #216

Closed maridonkers closed 1 year ago

maridonkers commented 1 year ago 
(define-format-all-formatter fourmolu
  (:executable "fourmolu")
  (:install "stack install fourmolu")
  (:languages "Haskell" "Literate Haskell")
  (:features)
  (:format (format-all--buffer-easy executable "--stdin-input-file" (buffer-file-name))))

But ormolu doesn't have the --stdin-input-file configuration (I think it needs it; see tweag/ormolu/issues/942#issuecomment-1328113066)

lassik commented 1 year ago

why does ormolu config not have the --stdin-input-file, like fourmolu has?

The formatter definitions were probably sent by different people, and we just forgot to add the flag to ormolu. Can you add it, test that it works, and if it does, send a PR?

maridonkers commented 1 year ago

PR: lassik/emacs-format-all-the-code/pull/217