Page MenuHomeSoftware Heritage

loader.cli: Add 2 subcommands, one to load, one to introspect
ClosedPublic

Authored by ardumont on Dec 11 2019, 4:31 PM.

Details

Summary

2 subcommands:

  • one to load: swh loader run <type> <url> <options>
  • one to list supported loader or instrospect the loader's extra arguments (most of the time url should be enough but not always) swh loader list swh loader list <type>

Related to T2134

Test Plan

tox

Diff Detail

Repository
rDLDBASE Generic VCS/Package Loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Dec 11 2019, 4:31 PM

lgtm

swh/loader/cli.py
89

import at the top of the file please

ardumont added inline comments.Dec 12 2019, 11:00 AM
swh/loader/cli.py
89

mmmph, yeah!

ardumont updated this revision to Diff 8602.Dec 12 2019, 11:02 AM

Move import statement at the top of the file

ardumont edited the summary of this revision. (Show Details)Dec 12 2019, 11:13 AM
ardumont updated this revision to Diff 8603.Dec 12 2019, 11:14 AM
  • loaders: Add missing class loader docstrings
  • test_cli: Test the cli list's standard use
douardda accepted this revision.Dec 12 2019, 11:17 AM

Nice, thx

This revision is now accepted and ready to land.Dec 12 2019, 11:17 AM