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

Replace simple dependency and drop 12 sub-deps 🎉 #62

Closed
wants to merge 2 commits into from
Closed

Replace simple dependency and drop 12 sub-deps 🎉 #62

wants to merge 2 commits into from

Conversation

fregante
Copy link

Faster and lighter? Sign me up!

https://www.npmjs.com/package/fast-deep-equal

deep-equal is downloading a lot of dependencies: https://npm.broofa.com/?q=bonjour

@mdidon
Copy link

mdidon commented Apr 6, 2021

@fregante We have implemented in https://github.com/onlxltd/bonjour-service if this is of any use.

@jhurliman
Copy link

@mdidon great to see active development on a successor project? Does it address #51 by chance?

@mdidon
Copy link

mdidon commented May 15, 2021

@jhurliman Thanks for bringing this to my attention.
Not yet, we're working through the issues/PRs listed under the original repo but I'll have a look into it next week.

@fregante
Copy link
Author

Friendly ping @watson. 😃 I know this repo is dormant, but the 2 packages are equivalent and would be a pretty easy improvement.

@toastal
Copy link

toastal commented May 28, 2022

In April 2022, Node.js v12.x was EoL as well. Might as well move the goal post to v14.x for latest maintained LTS. It would be a welcome update to get better performance and a smaller bundle with less dependencies for teams to audit. This project is being used in the massive project Quill and webpack-dev-server which would see a benefit upstream.

@fregante fregante closed this Jul 31, 2022
@fregante fregante deleted the patch-1 branch July 31, 2022 17:25
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

Successfully merging this pull request may close these issues.

4 participants