-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Discussion: Refactor/Rewrite/Replace some parts of this library #855
Comments
|
Would transition to TypeScript be considered too? I could help with that. |
Hey I would like to work on replacing the But the problem is what about the |
|
I suggest https://github.com/telegraf/client. It's typed, and should be close to supporting https://grammy.dev/advanced/transformers.html. |
Agree with @wojpawlik, high time this library used a lite client like Maybe even use middleware composition instead of event emitters. |
Related to #1007. |
Do you have any news about migrating this amazing package to more lightwight?
|
This library has some deprecated libraries e.g.
request
. Node.js has also progressed since when the core of this library was written; It has support for native promises/Async await e.t.c.Some of the proposals are:
Replace the bloated and deprecated
request
library with a more light weight alternative. The current proposals include:ethanent/phin, node-fetch/node-fetch, matthew-andrews/isomorphic-fetch, sindresorhus/got
.The replacement library should support most of whatrequest
supported.Potentially replace
Event Emitter 3
with a better drop in replacement likemcollina/mqemitter
or nativeEvent Emitter
. This needs more research.Replace
bluebird
with native promises/async-await.Support Node.js v12+ as per LTS schedule. Any feature/polyfill supporting < v12 should be removed.
Replace
pump
withpipeline
.Feel free to make your own proposals.
The text was updated successfully, but these errors were encountered: