-
-
Notifications
You must be signed in to change notification settings - Fork 110
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 is-plain-obj
deps
#248
Comments
Hey Anthony! And then? Do you use unified? How far do you want to go? Do you want to inline every dependency? Please at least put some time in your issue and respect our time too. There are of course alternatives and trade offs. |
How well does your code work? I’d assume it might break across realms, due to the |
Hello, thanks for the fast answer. To be honest, no I don't use unified. I'm just trying to help clean up the js ecosystem even if my contribution is a grain of salt. "Inline every dependency" isn't what we're trying to do, but some packages shouldn't exist. Take a look at this PR where a contributor saves tons of data by removing This oneline comes from : https://github.com/es-tooling/module-replacements/blob/main/manifests/micro-utilities.json If you don't want to change it that's fine, don't worries. |
I personally don’t really think this check is very useful: Lines 1254 to 1261 in 35e72b9
But I don’t think there’s a way around this check: Line 1165 in 35e72b9
I agree removing useless dependencies is a good thing. I don’t necessarily think this is a completely useless dependency. You should look at why a dependency is used and come up with a proposal to replace it. A suggestion to “remove dependency |
It’s used in case 1 because the dep is already there. The dep is there because of case 2. |
If there is a runtime performance reason, runtime correctness reason, or security reason to drop the dependency I'm all for it. The premise of minifying code or documentation to save bandwidth downloading to a developer machine is not one I subscribe to. For folks who are concerned with the bandwidth, take it up with CI/CD providers and NPM itself to offer more tailored download options, rather than going after individual projects over a few kilobytes. |
Thanks for your feedbacks, I see that we have different opinions about such packages (even within e18e) I don't want to get into a drama/debate because that's not the aim of this initiative. If you want to help or suggest things, I recommend you go to the discord. Thank you, all the best for your projects and sorry for the time lost. |
Initial checklist
Problem
https://e18e.dev/guide/cleanup.html
https://packages.ecosyste.ms/registries/npmjs.org/packages/is-plain-obj/dependent_packages?order=desc&sort=downloads
Unified is top 2 dependant of
is-plain-obj
Solution
Replace by a oneline function :
Alternatives
.
The text was updated successfully, but these errors were encountered: