Require 'os' in _nvm_node_info for older nodes

jorgebucaran / nvm.fish

The Node.js version manager you'll adore, crafted just for Fish

https://git.io/nvm.fish

MIT License

2.08k stars 69 forks source link

Closed myw closed 3 years ago

myw commented 3 years ago

Older nodes (e.g. node 4), error out when calling _nvm_node_info, since os isn't imported by default. Requiring it makes sure it's around.

jorgebucaran commented 3 years ago

Nice catch! Thanks!