Ben Gubler
Ben Gubler
@slorber, I'd love to hear your thoughts about this proposal since Docusaurus is one of the main users of npm-to-yarn.
@Jay-Karia I think for ease of use, it should detect whether or not it is an executor command (test for `npx|pnpm dlx|...`) automatically rather than requiring an extra argument flag.
@Jay-Karia thanks for the great work! I think we should probably update to a major version. Do you think this is ready to merge?
I just merged this (with the CLI, not through the web client) and released in https://github.com/nebrelbug/npm-to-yarn/releases/tag/v3.0.0! Thanks for your help!
@Jay-Karia I appreciate this PR, but as of right now I'm not convinced that it adds enough value to the package to make it worth the increased size and maintenance....
That's fascinating! To be honest, I'm not surprised -- useWith has been recommended against for a while, so it's likely not optimized by most common JS runtimes.
@bribes fascinating! I probably don't have time to test it right now, but it would be great if you wanted to add it to the documentation!
I just updated the docs to remove that suggestion. @bribes thanks for the heads up.
Fixed the docs!
@aranor01 thanks for the PR! Your reasoning makes sense, but I'm not sure how common a use case this is. In the meantime, you can just put that value in...