Any news on this one? Will this be merged? :)

Vennlig hilsen
Jostein Kjønigsen

jostein@kjonigsen.net 🍵 jostein@gmail.com
https://jostein.kjønigsen.no
On 1/28/23 15:28, Jostein Kjønigsen wrote:

On 1/28/23 08:23, Eli Zaretskii wrote:

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.
Fair enough!

- adding it in compile.el, to improve ability to oversee all
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?
I don't see why not.

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