Any news on this one? Will this be merged? :)
On 1/28/23 08:23, Eli Zaretskii wrote:
Fair enough!
I don't think doing this in the major-mode file will improve locality,
because we have compilation-minor-mode, which should be able to do its
thing even if the relevant major mode is not yet loaded.
- adding it in compile.el, to improve ability to oversee allI don't see why not.
expressions, and be able to optimize those?
I see csharp-mode.el has the expressions added directly there. Should I
go about preparing patches doing
the same for typescript-ts-mode too?
Ok. Attached is a patch which adds Typescript tsc-support to compile.el. Is this OK to install in emacs-29?
I also see in retrospect that my comment about csharp-mode above may been somewhat ambiguous and easy to misunderstand (and seemingly you did).
To be clear: csharp-mode includes the compilation-mode regexps in the major-mode, not in compile.el. Is this also something we should aim to fix for emacs-29, or should we leave that for master?
--
Jostein