Re: Flymake refactored

[Mark Oteiza]

Lele Gaifax <address@hidden> writes:

> Mark Oteiza <address@hidden> writes:
>
>> A small patch on top of it: this facilitates configuring the checker to
>> instead be flake8, which is a wrapper around a few checkers, including
>> pyflakes
>
> Yep, I like it to be flexible enough to do that, for example to be able to map
> "style issues" reported by the pycodestyle to ":note" entries... so your
> msg-alist seems better.
>
> Can you help me on a better definition for the following? In particular, is it
> possible to dynamically set the :value-type to allow only one of the keys in
> the `flymake-diagnostic-types-alist'?
>
>   (defcustom python-flymake-msg-alist
>     '(("\\(^redefinition\\|.*unused.*\\|used$\\)" . :warning))
>     "Alist used to associate messages to their types.
>   Each element should be a cons-cell (REGEXP . TYPE), where TYPE must be
>   one defined in the variable `flymake-diagnostic-types-alist'."
>     :group 'python-flymake
>     :type '(alist :key-type (regexp) :value-type string))

Oops, forgot about the type.  It can probably be

  :value-type (choice (const :error) (const :warning) ...)