New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

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

Open

doublep wants to merge 1 commit into gonewest818:master from doublep:public-no-color

Contributor

doublep commented Oct 30, 2021

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.


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

1c9796c

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet