Closed FurqanHabibi closed 1 year ago
@hanscg Just checking, is it ok to merge this in?
@stupoid It looks good to me, but I don't exactly manage this repo 🙇
@ojii Any comments? Otherwise I'll merge this
@ojii Any comments? Otherwise I'll merge this
I'm a little unhappy about setting an upper bound on urllib3 instead of a lower one but if this is needed, i don't have an objection to merging it.
Let's merge and see if we can do anything about urllib later 🙏
Dependency resolution took a long time because poetry chose the wrong version of urllib3. Specifying the urllib3 version solve the resolution.