motform / arduino-cli-mode

emacs support for the arduino-cli

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Improve doc

Ergus opened this issue · comments

Hi:

Just a suggestion. In the use-package snippet you propose in the readme it is simpler to use this:

(use-package arduino-cli-mode
  :ensure t
  :mode "\\.ino\\'"
  :custom
  (arduino-cli-warnings 'all)
  (arduino-cli-verify t))

BTW: Does this mode conflicts with the arduino-mode?

https://github.com/stardiviner/arduino-mode

If not, maybe it is better to use a hook instead of the mode to have both.

That use-package declaration looks a lot more succinct, do you want to submit a pull request to swap it out or should I?

arduino-cli-mode is a minor mode and should not conflict with arduino-mode, which is a major one. I use both side by side, but there could be some conflict somewhere that I have not encountered. Please report any if you find them!

Hi, feel free to do it yourself.

If the config works with arduino-mode properly, then the config must be:

(use-package arduino-cli-mode
  :ensure t
  :hook arduino-mode
  :custom
  (arduino-cli-warnings 'all)
  (arduino-cli-verify t))

And let the arduino-mode have the :mode

Cool, that looks great! I really have to dig into use-package some time. I have updated the README and will close this thread. : )