Page MenuHomeSoftware Heritage

Several fixes in npm, cgit and phab listers
ClosedPublic

Authored by douardda on Sep 3 2019, 12:03 PM.

Details

Summary

This diff consists in 3 commits:

  • cgit: fix cgit's task module and tests forgot some url_prefix there.
  • phabricator: fix the FullPhabricatorLister task

    forgot the forge_url -> api_baseurl renaming in there.
  • npm: make NpmVisitModel use the main declarative base class from core.models

    This is needed by the (refactored) db init mechanism, since this later uses the main declarative base class (thus the main MetaData instance) to gather tables to be created/dropped.

Diff Detail

Repository
rDLS Listers
Branch
plugins
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 7598
Build 10882: tox-on-jenkinsJenkins
Build 10881: arc lint + arc unit

Event Timeline

ardumont added a subscriber: ardumont.

ok with those fixes.

The ci is still red with errors in the swh.lister.cli.get_lister method (relative to the constructor refactoring as well).
My guess is that's not an issue since it will get deprecated by the plugin refactoring.

This revision is now accepted and ready to land.Sep 3 2019, 1:13 PM

Fix cli.py for test_cli to pass (phabricator)

This revision was automatically updated to reflect the committed changes.