Jostein Kjønigsen

Results 625 comments of Jostein Kjønigsen

Shouldn't you be using the tide-specific eldoc-function you just sent a PR for? 😅 **Edit:** Also no need to advice a function we already own and control. Better to just...

It's merged now, so at this point it's *required* to use 👍 I really don't know enough about this scenario to do a solid review, but if you submit another...

Good question. Looking into it, it seems the code-section involved for this [is fairly small](https://github.com/ananthakumaran/tide/blob/master/tide.el#L734-L740). My initial assumption was that [`tide-command:signatureHelp`](https://github.com/ananthakumaran/tide/blob/master/tide.el#L702-L708) was responsible for this. And as far as I...

> That's why I switched to lsp, it's been working better than tide tbh Disregarding everything else about this issue, I can really sympathize with this, and it's been a...

> The number of linters a user may run into from project to project is unlimited. While maybe not unlimited, it's clearly non-zero and more than one. And that's when...

Thanks for the thorough and detailed reply @lddubeau. Greatly appreciated. The current situation (at least from a code-sanity perspective) honestly sounds about as sad and depressing as it could possibly...

> 2. Provide an eslint equivalent to tide-add-tslint-disable-next-line. (I actually already have code for this.) I guess that leaves you @lddubeau to be a man of your word? :smile:

Take care of yourself and take your time. Certainly do NOT worry about silly GitHub issues! :laughing: I hope all goes well and wish you the speediest of recoveries! :heart:

I guess that means that if anyone else wants to take a stab at this, any PR is greatly welcomed :)

I saw this too in the TS3.5 announcement and was conflicted about creating an issue for it or not. Looking at the `expand-region` code though, it seems to provide mode-specific...