Page MenuHomeSoftware Heritage

Refactor the journal client using the new style functions.
Needs ReviewPublic

Authored by vlorentz on Fri, Jun 7, 4:15 PM.

Details

Reviewers
douardda
Group Reviewers
Reviewers
Summary

Depends on this swh-journal diff: D1563

Diff Detail

Repository
rDCIDX Object indexer
Branch
refactor-journal-client
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6206
Build 8565: tox-on-jenkinsJenkins
Build 8564: arc lint + arc unit

Event Timeline

vlorentz created this revision.Fri, Jun 7, 4:15 PM
vlorentz edited the summary of this revision. (Show Details)Fri, Jun 7, 4:24 PM
vlorentz updated this revision to Diff 5152.Fri, Jun 7, 4:32 PM

bump dep on swh-journal.

douardda requested changes to this revision.Thu, Jun 13, 2:20 PM
douardda added a subscriber: douardda.
douardda added inline comments.
swh/indexer/cli.py
186

What is this new command supposed to do? Shouldn't it come with a usage/description (docstring)? Is journal-client a good name for it?

This revision now requires changes to proceed.Thu, Jun 13, 2:20 PM
vlorentz updated this revision to Diff 5232.Fri, Jun 14, 10:33 AM

add docstring.

swh/indexer/cli.py
186

I couldn't find a better name. Suggestions welcome.