-
Notifications
You must be signed in to change notification settings - Fork 43
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
Is this package maintained? #129
Comments
Hey @danbars The biggest problem, was that for the last few months, I just didn't have much time allocated to continue adding new features to this package, or improve the documentation. So answering your questions, yes we intend to support wider use cases, as more teams internally at Cloudflare are now depending on it. |
Thanks for the comments @G4brym What I am confused about now is whether to use this package or Hono for routes on Cloudflare Workers? Many of the examples on Cloudflare's website now feature Hono, but in reading your comment, the teams internally at Cloudflare use this package and it will the default? I'd rather use something that is used internally at Cloudflare, as opposed to some other package, though it seems like the people who are building Hono also work at Cloudflare? |
Hey @osseonews the people who are building Hono also work at Cloudflare |
Thanks for the quick comment. I actually used to use itty-router, but then jumped to hono recently, because I thought that was the only thing Cloudflare was going to standardize on. So I appreciate you sharing this information. I think I might move back to itty-router now, as it is just much simpler to use. |
I see that PRs are generally merged, but there are quite a few open issues.
The statement is that this package is used in production in Cloudflare Radar.
Radar is fairly simple in terms of API complexity - it has quite a few endpoints, but all of them are just GET requests used to fetch data.
Many of the open issues evolve around more complex scenarios - different content types, empty responses, file upload, custom headers, etc.
So my question is this - is this the intention of the package owners to support wider use cases than what Radar needs?
Does the team actively investing time in it, or only if it happens to be needed by a feature in Radar?
I really like the package, but I prefer knowing upfront what I'm getting into.
Thanks
The text was updated successfully, but these errors were encountered: