-
Notifications
You must be signed in to change notification settings - Fork 2
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
Use flycheck-buffer-saved-p instead #2
Conversation
;; only saved files will be checked by mix | ||
(setq flycheck-check-syntax-automatically '(mode-enabled save)) | ||
(add-to-list 'flycheck-checkers 'elixir-mix)) | ||
(add-to-list 'flycheck-checkers 'elixir-mix) |
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.
Please don't remove flycheck-mix-setup
. Apart from breaking existing configurations, it's also bad style—loading Emacs Lisp libraries should never have side effects, i.e. change options, etc…
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.
thanks, wasn't sure about that. I was following https://github.com/flycheck/flycheck-mercury/blob/master/flycheck-mercury.el#L222 which was probably not a good example 😄
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.
@aaronjensen No, indeed. It should also provide a setup
function, imho.
7038db4
to
3b29740
Compare
@tomekowal Will you review this PR? |
Thanks, looks great 👍 |
`flycheck-mix` now uses `flycheck-buffer-saved-p` so it is safe to flycheck on idle: tomekowal/flycheck-mix#2
`flycheck-mix` now uses `flycheck-buffer-saved-p` so it is safe to flycheck on idle: tomekowal/flycheck-mix#2
`flycheck-mix` now uses `flycheck-buffer-saved-p` so it is safe to flycheck on idle: tomekowal/flycheck-mix#2
`flycheck-mix` now uses `flycheck-buffer-saved-p` so it is safe to flycheck on idle: tomekowal/flycheck-mix#2
Fixes #1
According to @lunaryorn, this is the typical way to ensure that the flychecker is only run on save.