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

Remove Tornado dependency #479

Open
JustAnotherArchivist opened this issue Sep 6, 2023 · 1 comment
Open

Remove Tornado dependency #479

JustAnotherArchivist opened this issue Sep 6, 2023 · 1 comment

Comments

@JustAnotherArchivist
Copy link
Contributor

Tornado used to run the show here, but it's only used for very few things now thanks to asyncio. Let's evaluate whether any of them really need Tornado, although I'd expect it can all be done without much fuss with the standard library. Getting rid of the dependency would help maintainability; we're currently stuck two major versions behind upstream.

@HeliosLHC
Copy link

I upgraded my wpull fork to the latest Tornado 6 a couple months back. I do have to agree it would be better if we perhaps remove it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants