Page MenuHomeSoftware Heritage
Differential D412

Add trivial command line for the PyPI loader
ClosedPublic
Actions

Authored by olasd on Sep 18 2018, 11:19 AM.

Details

Reviewers
ardumont
Commits
rDLDPY681b9f24749b: Add trivial command line for the PyPI loader
Summary

Very helpful for debugging

Depends on D411

Test Plan

python3 -m swh.loader.pypi devpi-common does something useful

Diff Detail

Repository
rDLDPY PyPI loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd created this revision.Sep 18 2018, 11:19 AM
Harbormaster completed remote builds in B1395: Diff 1285.Sep 18 2018, 11:19 AM
ardumont accepted this revision.Sep 18 2018, 11:23 AM
ardumont added inline comments.
swh/loader/pypi/loader.py
259

Nice.

Do we entertain that in the readme? [1]

[1] https://forge.softwareheritage.org/source/swh-loader-pypi/browse/master/README.md$87-102

This revision is now accepted and ready to land.Sep 18 2018, 11:23 AM
olasd updated this revision to Diff 1286.Sep 18 2018, 11:43 AM

Update README as well

Harbormaster completed remote builds in B1396: Diff 1286.Sep 18 2018, 11:43 AM
Closed by commit rDLDPY681b9f24749b: Add trivial command line for the PyPI loader (authored by olasd). · Explain WhySep 18 2018, 11:45 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
31 lines
swh/
loader/
pypi/
16 lines

Diff 1287

View Options

README.md

Loading...
View Options

swh/loader/pypi/loader.py

Loading...
About · Developer information · Legal