search

serverless / inquirer

inquirer with enforced serverless theme
MIT License
1 stars 2 forks source link

Uses `createRequire` when possible #4

Closed arcanis closed 3 years ago

arcanis commented 3 years ago

The documented createRequire function is available since Node 12 and lets you resolve module on behalf of other modules - which I believe is why you used ncjsm here. Since it's part of the Node API, it's expected to work regardless how the project is installed.

Fixing this (and updating serverless to reference an updated release) fixes #3.

arcanis commented 3 years ago

Interestingly, I just noticed that the package I modified locally on my machine was @serverless/utils, not @serverless/inquirer. I suspect those got merged together, and this repo is outdated?