Daniel Cousens
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:
Squashed and rebased on `main`
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