Wrap all invocations of `npm` in `node` to set the `PATH` appropriately

LuxDL / DocumenterVitepress.jl

Documentation with Documenter.jl and VitePress

https://luxdl.github.io/DocumenterVitepress.jl/

MIT License

62 stars 9 forks source link

Wrap all invocations of `npm` in `node` to set the `PATH` appropriately #123

Closed asinghvi17 closed 1 month ago

asinghvi17 commented 2 months ago

Fix #122

@korsbo, could you test this PR out and see if it works?

korsbo commented 2 months ago

I'll try, but I only trigger this error on CI and, for some reason, it does not trigger consistently, so it'll take longer to test than I have time for today (daycare closes soon).

korsbo commented 1 month ago

It seems to work! I've tried triggering the bug a few times on my CI setup where the original issue came from. The problem has been a bit inconsistent before, but I've tried with this PR enough times for it to be unlikely for the streak of CI passes to be a fluke. 🎉