search

gonewest818 / elisp-lint

Basic linting for Emacs Lisp
66 stars 7 forks source link

Make the variable added in commit bf72aee public: `elisp-lint-no-color' #33

Open doublep opened 3 years ago

doublep commented 3 years ago

This is a minor change suggested by me in #31, and also agreed to by the original author of ...-no-color change.

To quote:

Would be nice if elisp-lint--no-color was made public (i.e. elisp-lint-no-color) instead and settable (defvar instead of defconst). For any tool that integrates with the linter (in my case it's Eldev), it might be a desired thing to forward a higher-level setting to the linter. And it's always better to go via public interface and not meddle with private variables.