-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
The punycode
module is deprecated.
#41
Comments
My discord bot no longer works due to this package. I'm using mongodb as my database, and this package seems to be used to connect to it. So, I want you to fix this issue quickly. Can you do this? @domenic |
This package is a spare-time project. It's possible one of these weekends we'll make time for it. I would guess 2-5 weeks from now, looking at my current weekend schedule. Unfortunately, getting slammed with duplicate notifications actually makes me feel sad about the whole prospect of doing open source in my spare time, so is likely to lengthen the time until a fix is available since it makes me avoid GitHub on the weekends. If this is urgent for you, you can consider contracting me via email and we can discuss what kind of hourly consulting rate and payment plan would speed up the timeline for a fix. Until then, as I said, this is a spare time project and you should have no expectations of a quick fix. |
Thank you for your reply. |
I think it's already using npm punycode (not a core module) and this change fixes deprecation warnings: fef6e95 |
Yeah, I made time to test this issue and cannot reproduce any problems. require("tr46"); when using tr46 4.1.1. |
DeprecationWarning: The
punycode
module is deprecated. Please use a userland alternative insteadplease use punycode.js
The text was updated successfully, but these errors were encountered: