Convert empty author/committer name/email strings to bytes.
They were generated by the PyPI loader.
See also: swh-loader-pypi!23 (closed)
Migrated from D1773 (view on Phabricator)
Merge request reports
Activity
Build is green See https://jenkins.softwareheritage.org/job/DJNL/job/tox/201/ for more details.
Build is green See https://jenkins.softwareheritage.org/job/DJNL/job/tox/202/ for more details.
!44 (closed)#change-xD6YYPZMaH2q enables it
is that included in the coverage tooling we have on jenkins?
Relatedly to swh-environment#1411 (closed)
!44 (closed)#change-xD6YYPZMaH2q enables it
Ok...
Although i don't see how that's consistent with the migration we started from nose to pytest (swh-environment#1261 (closed)). Do we migrate to doctest now?
And if we choose both to have both (because why not), what's the rationale to decide what's written in what?
Do we migrate to doctest now?
No.
And if we choose both to have both (because why not), what's the rationale to decide what's written in what?
doctest are first and foremost part of the documentation, therefore they provide value to readers not familiar with the project (mostly as examples of how to use a function), so they must be easy to understand. eg. here they are all one-liners. Them being tests is a side-effect of being good (executable) examples of how to use the function
mentioned in merge request !245 (closed)
Build is green See https://jenkins.softwareheritage.org/job/DJNL/job/tox/213/ for more details.