alphapapa / makem.sh

Makefile-like script for linting and testing Emacs Lisp packages
GNU General Public License v3.0
163 stars 13 forks source link

Load elisp file before linting #24

Open noctuid opened 4 years ago

noctuid commented 4 years ago

Similar to #9 but for more complex situations, I'd like to be able to load a file with some configuration before linting happens. Here is an example. I'd also like to be able to fail CI runs if package-lint fails, but in order to do that, I need to be able to add advice to ignore unfixable errors.

If this seems reasonable, maybe look for a specific file name to avoid the need for user configuration (e.g. lint-config.el).

alphapapa commented 4 years ago

Generally that's more complicated than I want to get in this script. I feel like it's already more complicated than I planned. :)

However, I think you might be able to do this already: makem.sh loads Elisp files that either appear to be a test file or that (provide 'a-feature). So if you put something like (provide 'annalist-lint-init) at the bottom of that file, it should be loaded before the linting rule is run. I think that will cause its configuration to take effect in the subsequent linting.

If this does work, then I'd be happy to document it as an obviously intended consequence of the general design that clearly obviates the need for explicit configuration files and options... ;) And if it doesn't work, maybe a few adjustments would make it work.

Let me know! :)