From: Dmitry Gutov <dgutov@yandex.ru>
To: "João Távora" <joaotavora@gmail.com>
Cc: Mark Oteiza <mvoteiza@udel.edu>,
Lele Gaifax <lele@metapensiero.it>,
Stefan Monnier <monnier@iro.umontreal.ca>,
emacs-devel@gnu.org
Subject: Re: Three Flymake backends Was Re: Two issues with the new Flymake
Date: Mon, 13 Nov 2017 02:23:04 +0200 [thread overview]
Message-ID: <b037696f-4957-7f43-7929-7268f132cbc5@yandex.ru> (raw)
In-Reply-To: <871slbr7ft.fsf@gmail.com>
On 11/6/17 1:08 PM, João Távora wrote:
> Then probably a good idea to do what flycheck does because gremlins.
Sure.
>>> Hey I didn't spend more than 5 minutes with it :-)
>>
>> However long, I'm glad you did. Now, the version I've sent is pretty
>> much working.
>>
>> Will you push it to emacs-26?
>
> I think you should test it a tiny bit more and then you push it :-)
It seems to be working well, but I'm getting bogged down by minor details.
Do we add defcustoms for the program name and the config file name? Do
we really need the ruby-flymake-command defcustom? I doubt there is an
alternative program that gives the same output as 'ruby -wc'.
>> Where will it live?
>
> ruby-mode.el so it at least locally shares some code with ruby-flymake.
OK, code sharing will be step two.
>> How/when will Flymake choose between rubocop-flymake and ruby-flymake?
>
> Flymake can use both at the same time. Just
>
> (add-hook 'flymake-diagnostic-functions 'ruby-flymake nil t)
> (add-hook 'flymake-diagnostic-functions 'rubocop-flymake nil t)
>
> The user can remove-hook if he wants to.
>
> If rubocop does everything "ruby -w" does and more,
It does.
> then maybe a single
> backend. One that uses a ruby-flymake-use-rubocop-if-available
> defcustom, and then checks for (executable-find "rubocop").
Do we check for (executable-find "rubocop") once inside the major mode
function, or every time the checker is called?
In the latter case we'll have a function ruby-flymake-auto calling one
or the other.
prev parent reply other threads:[~2017-11-13 0:23 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-03 9:50 Two issues with the new Flymake Lele Gaifax
2017-11-03 12:33 ` Stefan Monnier
2017-11-03 14:07 ` Lele Gaifax
2017-11-03 16:59 ` João Távora
2017-11-03 17:15 ` Stefan Monnier
2017-11-03 20:17 ` Three Flymake backends Was " João Távora
2017-11-04 15:30 ` Stefan Monnier
2017-11-04 23:17 ` João Távora
2017-11-05 12:50 ` Dmitry Gutov
2017-11-05 12:59 ` João Távora
2017-11-05 13:04 ` Dmitry Gutov
2017-11-05 13:22 ` João Távora
2017-11-05 20:14 ` Dmitry Gutov
2017-11-05 21:05 ` João Távora
2017-11-05 23:56 ` Dmitry Gutov
2017-11-06 9:48 ` João Távora
2017-11-06 10:35 ` Dmitry Gutov
2017-11-06 11:08 ` João Távora
2017-11-13 0:23 ` Dmitry Gutov [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b037696f-4957-7f43-7929-7268f132cbc5@yandex.ru \
--to=dgutov@yandex.ru \
--cc=emacs-devel@gnu.org \
--cc=joaotavora@gmail.com \
--cc=lele@metapensiero.it \
--cc=monnier@iro.umontreal.ca \
--cc=mvoteiza@udel.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).