Page MenuHomeSoftware Heritage

package.archive: Move archive loader to its own namespace
ClosedPublic

Authored by ardumont on Dec 4 2019, 1:47 PM.

Details

Summary

This is pre-requisite work towards the unified cli.

(Other package loaders have been migrated the same way, i won't open similar diffs as this though, it's time consuming for everyone for little benefit)

Note:
This implies a scheduler's task_type migration on their backend name (P567)

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 4 2019, 1:47 PM
ardumont updated this revision to Diff 8433.Dec 4 2019, 3:35 PM

Reuse the same conftest configuration between tests (instead of duplication)

ardumont edited the summary of this revision. (Show Details)Dec 4 2019, 4:07 PM
ardumont updated this revision to Diff 8434.Dec 4 2019, 4:15 PM

Adapt setup.py accordingly

ardumont edited the summary of this revision. (Show Details)Dec 5 2019, 8:39 AM
ardumont edited the summary of this revision. (Show Details)Dec 5 2019, 9:13 AM
ardumont edited the summary of this revision. (Show Details)
douardda accepted this revision.Dec 5 2019, 12:04 PM
douardda added a subscriber: douardda.

LGTM

This revision is now accepted and ready to land.Dec 5 2019, 12:04 PM
ardumont updated this revision to Diff 8442.Dec 5 2019, 12:30 PM
  • package.npm: Move npm loader to its own namespace
  • package.pypi: Move pypi loader to its own namespace
  • loader.package: Remove dead code
ardumont updated this revision to Diff 8443.Dec 5 2019, 12:31 PM

Diff against origin/master and plug to master branch