Do not use *Sync calls, use promises or callbacks

As titled:

Line 22 in fc14cd8

if (!fs.existsSync(dir)) {

.

(And in many other places)

@mcollina Removed all Sync calls...I hope this is the right way to do it!
Do you have any other suggestion? Or do you think that's okay?

There is still https://github.com/davidedantonio/create-fastify-app/blob/master/lib/utils.js#L90.

Don't do https://github.com/davidedantonio/create-fastify-app/blob/master/lib/utils.js#L29-L62. remove cb and just use async functions. You are calling cb() multiple times in there.

process.exit() is a top-level concern, and should not really be in a utils file (https://github.com/davidedantonio/create-fastify-app/blob/master/lib/utils.js#L26).

Do not promisify manually (https://github.com/davidedantonio/create-fastify-app/blob/master/lib/fs.js), use https://nodejs.org/dist/latest-v8.x/docs/api/util.html#util_util_promisify_original.

Thanks again for the suggestions! 🙏

Do you think it is right to keep the fs.js file or move all the promisified functions to their respective files?

I would remove it.

I've removed the fs file utils except for fileExists function that I moved to the utils.js file.
I need that this function return true or false value, and I don't think it's a good solution to rewrite the same function in all the files where I need it.

Good work!