refactor: replace promisify with fs/promises version on convert.js

[hereje]

refactor: replace make-dir with fs/mkdir

• replace functionality related to create a directory
• remove dependency make-dir no longer needed

Related to netlify/cli#3941

[kitop]

One small comment. And then I'm curious if there's a way to make the package.json have less changes. Since there's a lot because it's changing from V1 to V3 - would it be possible to have it keep the old format? It'd be nicer to change the format in a specific PR.

[hereje]

One small comment. And then I'm curious if there's a way to make the package.json have less changes. Since there's a lot because it's changing from V1 to V3 - would it be possible to have it keep the old format? It'd be nicer to change the format in a specific PR.

Sorry, My mistake.

I will do it through:

npm install --lockfile-version 1

[kitop]

@hereje thanks for the update! Unfortunately, seems like there's still a lot of additions to the package-lock.json, like adding webpack, source-map and others that were not there before.

I've tried the same change in the package.json locally (remove mk-dir and add the engines definition) and here's the diff I get: https://gist.github.com/kitop/9b53d60e4b2650bc0a03f6036ac76477

Any reason why the other dependencies where added? I'd like to keep the package-lock.json as clean as possible.

[hereje]

@hereje thanks for the update! Unfortunately, seems like there's still a lot of additions to the package-lock.json, like adding webpack, source-map and others that were not there before.

I've tried the same change in the package.json locally (remove mk-dir and add the engines definition) and here's the diff I get: https://gist.github.com/kitop/9b53d60e4b2650bc0a03f6036ac76477

Any reason why the other dependencies where added? I'd like to keep the package-lock.json as clean as possible.

Yeah, it seems I poluted the file when I made a dependency install.

I have fixed the issue.

[netlify]

✅ Deploy Preview for open-api ready!

Name	Link
🔨 Latest commit	551cbc0
🔍 Latest deploy log	https://app.netlify.com/sites/open-api/deploys/65a5671930912d0008d1d6d3
😎 Deploy Preview	https://deploy-preview-522--open-api.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.