Daniel Cousens

Results 571 comments of Daniel Cousens

Still looking to do this? Just noticed `unorm` accounts for 15% of my applications build size haha.

Unfortunately `tsserver_path` isn't the only thing, `npm` is needed [elsewhere](https://github.com/pmizio/typescript-tools.nvim/blob/master/lua/typescript-tools/tsserver_provider.lua#L74-L84) A temporary workaround (that worked for me on Arch Linux) is to do the following https://github.com/pmizio/typescript-tools.nvim/issues/151#issuecomment-2664526140 - that helped me...

@pmizio is `npm_global_path` required, or can we make it optional if `tsserver_path` is set?

Likely superseded by https://github.com/keystonejs/keystone/pull/9253

Superseded by #9253, and if not, we'll re-open

I can't review this this week @iamandrewluca, but this looks amazing! :yellow_heart:

I am really open to accepting this work, my only concern is that we're trying to solve a number of problems in one pull request. If we can break this...

Can you please put any code moving operations into their own commits, so I can easily review those commits and then focus only on the diff?

@marekryb OK, if you can get everything into a working order, as atomic as reasonable, I could jump in towards the end