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 dependencies #304

Open
ejizba opened this issue Oct 9, 2024 · 0 comments
Open

Remove dependencies #304

ejizba opened this issue Oct 9, 2024 · 0 comments

Comments

@ejizba
Copy link
Contributor

ejizba commented Oct 9, 2024

We currently have 3 dependencies - undici, long, and cookie. Removing these would help eliminate concerns about npm audit alerts (e.g. #301) and improve cold start (tracked internally by this work item).

Unfortunately, I believe this would be a breaking change so we shouldn't do it until we're ready to do v5 of our npm package. Granted, I think this issue by itself could be reason enough to do a v5.

I already have some changes prepared and did some testing. I have a branch to remove each individual package (see the sub-issues), but here's a branch that removes all 3: ej/removeDependencies

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

No branches or pull requests

1 participant