Brett C. added the comment: Ezio seems to be right about user_id and getting a list or a name depending on which branch is taken based on the line that sets user_id initially. It also might be clearer if you check for `not len(user_id)` or `len(user_id) == 0` instead of `len(user_id) < 1`.
And thanks for handling the case when there's no match. I wouldn't be surprised if people don't sign the CLA but still get a PR merged that e.g. fixes a single line of the documentation. _______________________________________________________ PSF Meta Tracker <metatrac...@psf.upfronthosting.co.za> <http://psf.upfronthosting.co.za/roundup/meta/issue611> _______________________________________________________ _______________________________________________ Tracker-discuss mailing list Tracker-discuss@python.org https://mail.python.org/mailman/listinfo/tracker-discuss Code of Conduct: https://www.python.org/psf/codeofconduct/
