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

Deprecating old ipc option #1056

Open
ehmicky opened this issue May 14, 2024 · 1 comment
Open

Deprecating old ipc option #1056

ehmicky opened this issue May 14, 2024 · 1 comment

Comments

@ehmicky
Copy link
Collaborator

ehmicky commented May 14, 2024

We've replaced the {stdio: [..., 'ipc']} option with the simpler {ipc: true}. The old syntax has been undocumented. However, it's still currently supported, for backward compatibility.

In the next major release, should we remove support the old syntax? Possible options:

  1. Keep as is: undocumented only
  2. Remove the types only
  3. Remove the types, and also throw an error message when using that old syntax at runtime

Which one do you prefer @sindresorhus?

Note: we probably should wait for many months before doing any new major release. We just did a big major release with many breaking changes. So this issue is for future planning, not immediate action.

@sindresorhus
Copy link
Owner

I would do 3.

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

2 participants