use treq instead of requests

pixelated / pixelated-user-agent

User facing components of Pixelated: a JavaScript single page app and a RESTful service.

GNU Affero General Public License v3.0

159 stars 72 forks source link

use treq instead of requests #1060

Open kalikaneko opened 7 years ago

kalikaneko commented 7 years ago

in bitmask we removed the requests dependency some time ago. we recently introduced treq, since it has a nice api and simplifies a lot all the http client code.

I think pixelated could depend on treq instead of requests, it also plays nicely with the rest of the async codebase.

tuliocasagrande commented 7 years ago

Hey @kalikaneko, we use requests marginally on some auxiliary resources, but I agree with you it'd be suitable to move everything to treq.

I'm afraid we won't be able to prioritize this at this moment, though. On the other hand, if you submit a PR with it, I'll be happy to review it.