-
Notifications
You must be signed in to change notification settings - Fork 206
Added support for user configuration #1079
base: master
Are you sure you want to change the base?
Conversation
…hie.yaml’. See ‘./docs/hie-yaml-file.md’ for details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could these options be added to the LSP configuration settings, instead of creating a new file/format?
haskell-ide-engine/hie-plugin-api/Haskell/Ide/Engine/Config.hs
Lines 21 to 29 in 7dd6fd7
data Config = | |
Config | |
{ hlintOn :: Bool | |
, maxNumberOfProblems :: Int | |
, diagnosticsDebounceDuration :: Int | |
, liquidOn :: Bool | |
, completionSnippetsOn :: Bool | |
, formatOnImportOn :: Bool | |
} deriving (Show,Eq) |
Certainly. I’ve just implemented the necessary changes for using the preexisting settings format, and updated the So as a workaround if the value isn’t available it defaults to checking for a I suppose the name of this branch doesn’t make any sense now. I wasn’t sure what to rename it to so I left it as is. |
The It is also not clear to my whether the |
@colbyn The settings.json location depends on the client, for LanguageClient-neovim it should be in |
This is something I added a little while ago and thought others may find this useful (includes a doc file
./docs/hie-yaml-file.md
).The primary feature is to just simply disable interactive (as you type) analysis, as mentioned in the
./docs/hie-yaml-file.md
file. For a reference the RLS project has a similar option calledbuild_on_save
(https://github.com/rust-lang/rls#configuration
).