-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
From evol262:
AwfulClient.userinfo is a huge mess of shit that should be in UserInfo, which should inherit from AwfulClient and set all that in init. AwfulClient.userinfo can return UserInfo(userid) which sets it all
Everything you're doing with "contacts_elem.find('dt'..." could (and arguably should, so it's testable) be done from a private helper method that takes **kwargs if necessary to pass extra shit in. How many times is "elem.find(...).get_text()" in there?
Same for "[tag.extract() for tag in soup.findAll(...)]"
And in __process_paginators
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels