Skip to content

Changes in hlint config file should be applied on the fly #594

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jneira opened this issue Nov 12, 2020 · 1 comment
Open

Changes in hlint config file should be applied on the fly #594

jneira opened this issue Nov 12, 2020 · 1 comment

Comments

@jneira
Copy link
Member

jneira commented Nov 12, 2020

(from twitter: https://twitter.com/korayal/status/1326963429257711623)

  • I describe that behaviour prematurely in docs but it does not work
  • We should change getHlintSettingsRule from defineNoFile to a definition that takes in account possible configuration files to rerun the rule

getHlintSettingsRule :: HlintUsage -> Rules ()
getHlintSettingsRule usage =
defineNoFile $ \GetHlintSettings ->
liftIO $ case usage of
HlintEnabled cmdArgs -> argsSettings cmdArgs
HlintDisabled -> fail "hlint configuration unspecified"

@Profpatsch
Copy link

Seconded, I run into this every time I change the hlint file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants