Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove execaCommand() #1055

Open
ehmicky opened this issue May 14, 2024 · 0 comments
Open

Remove execaCommand() #1055

ehmicky opened this issue May 14, 2024 · 0 comments

Comments

@ehmicky
Copy link
Collaborator

ehmicky commented May 14, 2024

See #1016 and #1054

execaCommand() and execaCommandSync() have been deprecated in favor of parseCommandString(). Their documentation has been removed. However, their implementation and types have been kept for backward compatibility.

In the next major release, those methods should be completely removed.

As an internal cleanup task, we will also want to remove the file and commandArguments returned by the internal function mapArguments(), since this is only used by execaCommand().

const {
file = initialFile,
commandArguments = initialArguments,
options = mergedOptions,
isSync = false,
} = mapArguments({file: initialFile, commandArguments: initialArguments, options: mergedOptions});

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

No branches or pull requests

1 participant