Skip to content

Conversation

@PatrykKuniczak
Copy link
Contributor

@PatrykKuniczak PatrykKuniczak commented Dec 21, 2025

Overview

I fixed check for no argument for scripts

if (pkg == null) 

which don't work(because if user don't type nothing undefined is return
with

if(!pkg)

which works :)

And i cleaned code a little, of unnecessary console.log()

Manual Testing

Try to use all of those scripts without argument.

@netlify
Copy link

netlify bot commented Dec 21, 2025

Deploy Preview for creative-fairy-df92c4 ready!

Name Link
🔨 Latest commit 70fa9fd
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/694d13bd3c16650008b634d2
😎 Deploy Preview https://deploy-preview-2015--creative-fairy-df92c4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@PatrykKuniczak PatrykKuniczak changed the title fix(scripts): fix void warnings of fs-extra(readFile, writeFile) fix(scripts): fix void warnings of fs-extra(readFile, writeFile) and recognizing lack of argument for scripts Dec 22, 2025
@PatrykKuniczak PatrykKuniczak force-pushed the fix/warnings-and-errors-of-scripts branch from 358bfe1 to 3be4972 Compare December 24, 2025 13:10
@PatrykKuniczak PatrykKuniczak changed the title fix(scripts): fix void warnings of fs-extra(readFile, writeFile) and recognizing lack of argument for scripts fix(scripts): fix recognize lack of argument for scripts Dec 25, 2025
@PatrykKuniczak
Copy link
Contributor Author

Sorry for a little mess, i haven't checked rest of the code for fs-extra references, we need to fix it in another way, change fs-extra to fs/promises isn't solutions :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant