-
Notifications
You must be signed in to change notification settings - Fork 82
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
chore: Make router a python package #17
Comments
Hey, I saw the roadmap mentions a "more performant router using non-Python languages." If we plan to tackle that soon, maybe we could explore rewriting the router in another language for a performance boost and ignore this. Just a thought. |
Hey @gaocegege , making it a python package sounds great to me! We are right now working on end-to-end tests and CI pipeline. And it should be available soon. Thanks for pointing that out! |
Thanks for the prompt response. Should we turn it into a Python package now, or wait until the CI setup is ready? It may impact the CI implementation. |
It would be great if we could turn it into a python package now. (otherwise we need to adjust the CI again IIUC?) |
Making the router a Python package offers several benefits:
Besides, we should include some basic end-to-end (e2e) tests as part of the continuous integration (CI) pipeline.
The text was updated successfully, but these errors were encountered: