pagure-importer

#136 Don't add contributor's name if email is not found
Opened by batman. Modified
batman/pagure-importer error  into  master

Download 136.patch

All we look for while going through the commits of a
project is the emails and if we don't find that. There
is no point of entering their name

instead of having the continue you could test that commit.author or commit.author.email is not None
Edited by cverna

rebased

rebased

I think commit.author.email fails with fedora-infra/bodhi. pygithub is throwing an exception.
To be safe we could add a try except block around these tests

rebased

did you try it on bodhi ?

did you try it on bodhi ?

Yeah, it fails :/

if you add a try except block around this code it should work. I think it is acceptable to ignore the exception here.
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.