Page MenuHomeSoftware Heritage

cli: make the deposit cli command a subcommand of the main 'swh' one
ClosedPublic

Authored by douardda on Apr 19 2019, 2:38 PM.

Details

Summary

'swh' being the new master command group (defined in swh.core); so:

swh deposit --help

works. The 'admin' (deposit server management) is only available if the
swh.deposit[server] optional package is installed.

Ensure the 'swh-deposit' still works for now.

also rename the 'deposit' subcommand as 'upload'
to prevent the ugly 'swh deposit deposit' in favor of:

swh deposit upload

Depends on D1426.
Related to T1671.

Diff Detail

Repository
rDDEP swh-deposit
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Apr 19 2019, 2:38 PM
douardda edited the summary of this revision. (Show Details)Apr 19 2019, 2:39 PM
douardda edited the summary of this revision. (Show Details)
douardda updated this revision to Diff 4642.Apr 19 2019, 3:50 PM

Add requests to requirements.txt since it does come with swh.core any more

ardumont added inline comments.
setup.py
59 ↗(On Diff #4642)

Isn't it supposed to be swh.cli.subcommands?

douardda updated this revision to Diff 4693.May 6 2019, 1:39 PM

Use proper swh.cli.subcommands entrypoint

ardumont accepted this revision.May 6 2019, 1:46 PM
This revision is now accepted and ready to land.May 6 2019, 1:46 PM
douardda updated this revision to Diff 4703.May 6 2019, 3:31 PM

depends on swh.core 0.0.60

This revision was landed with ongoing or failed builds.May 6 2019, 3:34 PM
This revision was automatically updated to reflect the committed changes.